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.

Sign up for GitHub

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

Jump to bottom

vlan regex misses MANY 1000-3000, and vlans ending in [6-9] above 4000. #116

Closed

robbat2 opened this issue Jul 19, 2015 · 1 comment

Contributor

robbat2 commented Jul 19, 2015

The vlan match fails on vlans numbered 40[0-8][6-9]

Here's the buggy code:
vlan_range_regex = %r[\d{1,3}|40[0-9][0-5]]

Correct it to:
vlan_range_regex = %r[[1-3]?\d{1,3}|40[0-8]\d|409[0-5]]

Contributor

igalic commented Aug 7, 2015

fixed in #116

igalic closed this as completed

vide mentioned this issue

Partial VLAN support refactor #98

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment