Modify

Opened 3 years ago

Last modified 3 years ago

#18018 new defect

luci-app-ddns is missing the event interface option

Reported by: techstep@… Owned by: developers
Priority: normal Milestone:
Component: packages Version: Barrier Breaker 14.07
Keywords: luci-app-ddns Cc:

Description

version: luci-app-ddns - 0.12+svn-r10530-1

Without the event interface option updates are only
done using the static intervals.

Attitude Adjustment 12.09:

config service 'myddns'
        option interface 'wan'
        option check_interval '10'
        option check_unit 'minutes'
        option force_interval '72'
        option force_unit 'hours'
        option ip_source 'network'
        option ip_network 'lan'

Barrier Breaker 14.07

config service 'myddns'
        option ip_source 'network'
        option ip_network 'wan'
        option check_interval '10'
        option check_unit 'minutes'
        option force_interval '72'
        option force_unit 'hours'

Attachments (2)

aa.PNG (19.2 KB) - added by techstep@… 3 years ago.
openwrt_ticket_18018.patch (690 bytes) - added by chris5560 3 years ago.

Download all attachments as: .zip

Change History (8)

Changed 3 years ago by techstep@…

comment:1 Changed 3 years ago by chris5560

Attached patch can be used for luci-0.12 AND luci-trunk
I current luci-trunk is the same error.
It's an svn patch for luci svn

Christian

Changed 3 years ago by chris5560

comment:2 Changed 3 years ago by anonymous

This needs some changing anyway, in my understanding "interface" means just how/when the script is started the first time, it has nothing to do with the IP, it's like when there was no real init.d script (which isnt there now yet anyway). The only logic value is "lan" anyway, not wan, meaning, the script goes live at boot time, when lan comes up.

comment:3 Changed 3 years ago by chris5560

Since BB14.07 / LuCI 0.12 it's necessary to enable ddns in LuCI->System->Startup or to run /etc/init.d/ddns enable from the console. The event interface should be wan because it makes no sense to send ddns updates when lan comes up and wan is not up. wan is also the interface holding your external IP. option interface defines the interface (should be correct "network") where hotplug monitor up/down events and re-start the scripts (but only if enabled in configuration AND as service). So no other changes needed.

comment:4 Changed 3 years ago by techstep@…

Will the patch be included any time soon in bb/trunk ? Because I'm not sure I'll be able to compile it for myself.

comment:5 Changed 3 years ago by chris5560

It's a question of time.

Short term you can edit the file directly on the router. It's a script/text file.
So no need to recompile by yourself.
The file is /usr/lib/lua/luci/model/cbi/ddns/ddns.lua
Don't forget to make a backup-copy of the file before editing ;-)
Copy the 4 lines from the patch between line 28 and 29 of original file. Have a look at the patch.
That's it.

If this works for you. Please close the ticket.
Patch will be implemented as soon as possible
Christian

comment:6 Changed 3 years ago by techstep@…

I've applied the patch, the GUI option is back, ddns configuration gets written properly, dns updates working fine.

Thanks for the fast response and revolving the issue so quickly!
Ticket can be closed (I threw away my cookies so I can't do it myself).

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.