Sanitize hostnames, macs, and vendor names before running .lower/.upper on them to prevent AttributeError

[slammingprogramming]

Fixes #1088 by sanitizing the inputs to mac, vendor, and name before running .upper/.lower to ensure they are type STR to avoid AttributeError

Closes #1088

[jokob-sk]

Cheers @slammingprogramming 🙏🙂

[jokob-sk]

Hey @slammingprogramming 👋 do you think your changes to device_handling.py can be merged to main? Thinking of pushing out a release this week before I start working on #724

[slammingprogramming]

@jokob-sk the ones i have submitted should be fine, that other original one we reverted I don't recommend, only because I haven't confirmed all changes are stable. this one is stable, though. simple fix for the string sanitization is all. it's been working reliably for me with this fix injected into my docker of netalertx (the stable release docker)

i have not yet tested it working under all changes in the netalertx-dev docker, so I'd recommend we make sure it works with all changes made there. But it's probably fine.

[jokob-sk]

Thanks! Feel free to open a new PR once you are happy and I have a look 🙂