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

FS#4082 - New default 'Support-UDP-Traceroute' rule might have illegal character (Firewall or LuCI or both) #9061

Open
openwrt-bot opened this issue Oct 11, 2021 · 1 comment
Labels

Comments

@openwrt-bot
Copy link

Lantizia:

As mentioned on IRC/OFTC...

hmm there is a broken default 'Traffic' Rule' of 'Support-UDP-Traceroute' on 21.02
which is disabled by default... but wouldn't work even if you enabled it as the port is 33434:33689 rather than 33434-33689
is this reported anyplace? I could report it... but where would I?
Lantizia: bugs.openwrt.org is the place for reporting issues
we'd appreciate the report :) Thank you for finding and mentioning!

This new default firewall rule (although disabled by default) seems to have been introduced with commit de8b88c.

I'm not sure if the colon (rather than a hyphen) affects the possibility of this rule working if it is activated without using LuCI.

But certainly from within LuCI... if try to edit this rule and re-save it (without changing anything) then it'll flag the field as having invalid contents and refuse to save.

If colons are valid in this context then I'll re-file this bug to the LuCI bug tracker instead.

@Lantizia
Copy link

Just realised this is something I raised on the old bugtracker, but for some reason three alternate versions of this issue (all by me it seems) were imported by @openwrt-bot and then immediately closed? Perhaps this was me using the edit button on the old bugtracker to add more detail, and the migration took all the various versions?

So I'm not sure which is the most valuable but these are the alternate versions of this issue which I guess can be marked as dupes... #9130 #9131 #9132

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

No branches or pull requests

2 participants