Skip to content
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

Include all ethernet interface types used in the wild in the room interface listing #2283

Conversation

lunkwill42
Copy link
Member

  • Although RFC 3635 specifies that implementations MUST use ifType=ethernetCsmacd(6) for all ethernet-like interface types, many fielded implementations have incorrectly used other values that were never reserved by the IETF.
  • D-Link is one such vendor, where interfaces would not be listed in the room interface list because of this.

Fixes #2280

Because:

- Although RFC 3635 specifies that implementations MUST use
  ifType=ethernetCsmacd(6) for all ethernet-like interface types, many
  fielded implementations have incorrectly used other values that were
  never reserved by the IETF.
- D-Link is one such vendor, where interfaces would not be listed in the
  room interface list because of this.

Fixes Uninett#2280
@lunkwill42 lunkwill42 requested a review from hmpf May 10, 2021 13:54
@lunkwill42 lunkwill42 self-assigned this May 10, 2021
@lunkwill42 lunkwill42 deleted the branch Uninett:5.1.x October 19, 2021 12:17
@lunkwill42 lunkwill42 closed this Oct 19, 2021
@lunkwill42 lunkwill42 changed the title Include all ethernet interface types used in the wild in the room interface listing Include all ethernet interface types used in the wild in the room interface listing Oct 19, 2021
@lunkwill42
Copy link
Member Author

Well dammit. It was not my intention to have this PR closed, but since the old stable branch was removed, GitHub closed it automatically - and apparently removing all options to edit it and re-open it against a differently named branch...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants