netbox-community / Device-Type-Library-Import

This library is intended to assist with importing device and module types into NetBox from the NetBox Community DeviceType-Library
https://github.com/netbox-community/devicetype-library
MIT License
265 stars 82 forks source link

Fixes #34: Change Vendor Delimiter #41

Closed FloEisen closed 2 years ago

FloEisen commented 2 years ago

Changed the split-delimiter from empty (white spaces) to comma. Because with white spaces as delimiter all vendors with a white space in their name cannot be imported (quotes didn't work in ENV strings).

For example: docker run -e "VENDORS=Cisco Palo Alto " netbox-devicetype-import-library This would be trated as three vendors.

Instead with comma as delimiter: docker run -e "VENDORS=Cisco,Palo Alto" netbox-devicetype-import-library Would be treated as wo vendors like it should.

Merging this change means, that you have to use a comma separated vendor list instead of white space separated from now on. But all the vendors with white spaces in thier name will then be supported as well (tested it already).

Fixes #34