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

Client rules don't apply by default #34

Open
abstractionmage opened this issue Oct 19, 2022 · 3 comments
Open

Client rules don't apply by default #34

abstractionmage opened this issue Oct 19, 2022 · 3 comments

Comments

@abstractionmage
Copy link

The firewall rules created via luci-access-control's UI don't work by default on my router. Manually changing the automatically generated firewall rules' "Protocol" setting from the default value of "0" to "Any" seems to fix them (see demonstration in the gif screen capture below).

luci-acces-control not working

OpenWrt version: 22.03.2 r19803-9a599fee93
Router model: TP-Link WR841N version 13.1

@minax007
Copy link

minax007 commented Dec 9, 2022

@abstractionmage
Thank you very much for the excellent illustration on how to set it up correctly.
Now it works fine.

Nevertheless it would be good if @k-szuster could fix this issue.
Thanks in advance!

@AlicjaWC
Copy link

AlicjaWC commented Feb 1, 2023

If you have the possibility, I suggest experimenting with the file /usr/lib/lua/luci/model/cbi/acces_control.lua
I think the line 110 is responsible for this -- self.map:set(section, "proto", "0")
And if you can determine what value should be inserted instead of "0", give feedback. As I see the problem occurred in OpenWrt 22.03.

@ml1969
Copy link

ml1969 commented Feb 21, 2023

@AlicjaWC
I had the same problem on OpenWrt 22.03.0 r19685-512e76967f
I edited /usr/lib/lua/luci/model/cbi/acces_control.lua line 70
from
s_rule.defaults.proto = "0"
to
s_rule.defaults.proto = "any"
and it seems to work 👍

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

No branches or pull requests

4 participants