New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use full name for vendor_oui #15234
Use full name for vendor_oui #15234
Conversation
Reason I used the short name is due to column width in tables. Feel free to change it to store both short and long names and use in the appropriate spots. |
Ok. New PR in #15236 |
Trimmed variables to get rid of unnecessary blanks after split of the CSV. |
66816ee
to
c3ede34
Compare
Tested successfully, search working as well. Looks good from my pov. |
This pull request has been mentioned on LibreNMS Community. There might be relevant details there: |
* Use full name for vendor_oui * trimmed * typo * more spaces to remove * don't set $short_vendor * keep line return --------- Co-authored-by: Tony Murray <murraytony@gmail.com>
When code was reworked to store in DB, short name (field 2 of CSV) was used. This is incomplete (truncated). This PR restores the full registered IEEE name which is in the 3rd column of the CSV.
DO NOT DELETE THE UNDERLYING TEXT
Please note
Testers
If you would like to test this pull request then please run:
./scripts/github-apply <pr_id>
, i.e./scripts/github-apply 5926
After you are done testing, you can remove the changes with
./scripts/github-remove
. If there are schema changes, you can ask on discord how to revert.