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

ASN rules are case sensitive (requires uppercase 'AS' prefix) #1393

Closed
redactedscribe opened this issue Dec 17, 2023 · 1 comment
Closed
Labels
bug TYPE: a report on something that isn't working

Comments

@redactedscribe
Copy link

redactedscribe commented Dec 17, 2023

What happened:

An ASN rule requires it to use an uppercase AS otherwise the rule won't have any effect (ASN related IPs are red/blocked).

What did you expect to happen?:

Like all other rules I've encountered, internet tcp, LAN UDP/1337 etc, be effective regardless of case.

How did you reproduce it?:

as##### in Global Settings / Network Noise incoming rules, presumably the same issue for any profile. 'Force Block Incoming Connections' is disabled for Network Noise and enabled globally. Still, a lowercase as will not work. Uppercasing to AS##### solves the issue (ASN related IPs now show in green).

Thanks.

@redactedscribe redactedscribe added the bug TYPE: a report on something that isn't working label Dec 17, 2023
@dhaavi
Copy link
Member

dhaavi commented Dec 18, 2023

Hey @redactedscribe, thanks for reporting this.

Requiring the "AS" to be uppercase is intentional. Country codes also have to be uppercase.

"Internet", "LAN" and "Localhost" are case insensitive because they are uncommon keywords for firewalls.

@dhaavi dhaavi closed this as not planned Won't fix, can't repro, duplicate, stale Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug TYPE: a report on something that isn't working
Projects
None yet
Development

No branches or pull requests

2 participants