diff options
author | Jo-Philipp Wich <jow@openwrt.org> | 2008-09-30 20:53:46 +0000 |
---|---|---|
committer | Jo-Philipp Wich <jow@openwrt.org> | 2008-09-30 20:53:46 +0000 |
commit | 871ba2df73dd42ee9fa80533dd924f192f0b3b25 (patch) | |
tree | c8733c4bb21de2bb607cc680df2bdc101e7dd879 /applications/luci-fw/luasrc/model | |
parent | 3993bac38d0b3fd696808faf27726ec103fe911b (diff) |
* 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
Diffstat (limited to 'applications/luci-fw/luasrc/model')
-rw-r--r-- | applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua b/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua index 8ff03b02d..0d9af93fc 100644 --- a/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua +++ b/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua @@ -18,10 +18,10 @@ s.addremove = true s.anonymous = true iface = s:option(ListValue, "src") -iface:value("") iface.rmempty = true oface = s:option(ListValue, "dest") +oface:value("") oface.optional = true luci.model.uci.cursor():foreach("firewall", "zone", @@ -56,6 +56,7 @@ dport:depends("proto", "tcpudp") jump = s:option(ListValue, "target") jump.rmempty = true +jump.default = "ACCEPT" jump:value("DROP", translate("fw_drop")) jump:value("ACCEPT", translate("fw_accept")) jump:value("REJECT", translate("fw_reject")) |