Opened 6 years ago
Last modified 4 years ago
#10473 new defect
openvpn new uci config vs. luci luci-app-openvpn
Reported by: | Bogdan <abr28@…> | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | Barrier Breaker 14.07 |
Component: | luci | Version: | Trunk |
Keywords: | Cc: |
Description
In r29167 the openvpn uci config and init.d script have been upgraded to use new functions but also some new uci config, more specifically changing 'enable' to 'enabled'. This breaks the compatibility with the LuCI openvpn app since the latter is still looking for 'enable'.
The fix is easy, just change line 73 of /usr/lib/lua/luci/model/cbi/openvpn.lua to read "enabled" instead of "enable":
s:option( Flag, "enabled", translate("Enabled") )
Cheers
p.s. I did submit a bug to LuCI at http://luci.subsignal.org but am not 100% sure if the fix cannot be implemented here
Attachments (0)
Change History (3)
comment:1 Changed 6 years ago by Sven.Roederer@…
comment:2 Changed 5 years ago by Del <delonly@…>
This bug is fixed in trunk, please close the report.
comment:3 Changed 4 years ago by jow
- Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07
Milestone Attitude Adjustment 12.09 deleted
In r29355 a fix was made to accept "option enable" and "option enabled" in init-script.
But to have two options for one purpose may be confusing. Also there is still the possibility to start openvpn by init-script, even when it shows disabled in luci-app-openwrt.