Modify

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@…

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.

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

Add Comment

Modify Ticket

Action
as new .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.