* luci/app/firewall:
- remove empty value from src zone, param is mandatory - add empty value to outgoing zone to allow removal of this param - change default action to "accept" to prevent unwanted lockouts
This commit is contained in:
parent
3993bac38d
commit
871ba2df73
1 changed files with 2 additions and 1 deletions
|
@ -18,10 +18,10 @@ s.addremove = true
|
||||||
s.anonymous = true
|
s.anonymous = true
|
||||||
|
|
||||||
iface = s:option(ListValue, "src")
|
iface = s:option(ListValue, "src")
|
||||||
iface:value("")
|
|
||||||
iface.rmempty = true
|
iface.rmempty = true
|
||||||
|
|
||||||
oface = s:option(ListValue, "dest")
|
oface = s:option(ListValue, "dest")
|
||||||
|
oface:value("")
|
||||||
oface.optional = true
|
oface.optional = true
|
||||||
|
|
||||||
luci.model.uci.cursor():foreach("firewall", "zone",
|
luci.model.uci.cursor():foreach("firewall", "zone",
|
||||||
|
@ -56,6 +56,7 @@ dport:depends("proto", "tcpudp")
|
||||||
|
|
||||||
jump = s:option(ListValue, "target")
|
jump = s:option(ListValue, "target")
|
||||||
jump.rmempty = true
|
jump.rmempty = true
|
||||||
|
jump.default = "ACCEPT"
|
||||||
jump:value("DROP", translate("fw_drop"))
|
jump:value("DROP", translate("fw_drop"))
|
||||||
jump:value("ACCEPT", translate("fw_accept"))
|
jump:value("ACCEPT", translate("fw_accept"))
|
||||||
jump:value("REJECT", translate("fw_reject"))
|
jump:value("REJECT", translate("fw_reject"))
|
||||||
|
|
Loading…
Reference in a new issue