openwrt / telephony

The telephony packages feed
104 stars 256 forks source link

siproxd: option registration_file does not work #854

Open schuettecarsten opened 10 months ago

schuettecarsten commented 10 months ago

If I try to add option registration_file /opt/data/siproxd/siproxd-general.reg to siproxd config file, the setting is lost and not processed. Without a value for registration_file in config file, the generated siproxd-general.conf contains a default value for this setting, if I try to overwrite it, the whole setting is lost.

My /etc/config/siproxd (comments removed):

config siproxd general
        option registration_file /opt/data/siproxd/siproxd-general.reg
        option hosts_allow_reg 192.168.10.0/24
        option sip_listen_port 5060
        option if_inbound br-eth0.999
        option if_outbound pppoe-wan
        list load_plugin 'plugin_logcall.so'
        list load_plugin 'plugin_fix_bogus_via.so'
        option plugin_fix_bogus_via_networks 10.0.0.0/8,172.16.0.0/12,192.168.0.0/16

Generated /var/etc/siproxd/siproxd-general.conf:

# config auto-generated from /etc/config/siproxd
hosts_allow_reg = 192.168.10.0/24
sip_listen_port = 5060
if_inbound = br-eth0.999
if_outbound = pppoe-wan
load_plugin = plugin_logcall.so
load_plugin = plugin_fix_bogus_via.so
plugin_fix_bogus_via_networks = 10.0.0.0/8,172.16.0.0/12,192.168.0.0/16
autosave_registrations = 300
rtp_port_low = 7070
rtp_port_high = 7089
rtp_timeout = 300
rtp_dscp = 46
tcp_timeout = 600
tcp_keepalive = 20
default_expires = 600
daemonize = 0
user = nobody
plugindir = /usr/lib/siproxd/

If I comment out my option registration_file, the generated siproxd-general.conf looks like this:

[...]
user = nobody
registration_file = /var/lib/siproxd/siproxd-general.reg
plugindir = /usr/lib/siproxd/

I want to move the registration file to a persistant storage, which does not work at the moment. According to siproxd documentation (https://siproxd.tuxworld.ch/siproxd_guide.pdf), making the registration file persistent is a common scenario, so it should be supported by OpenWrt.

Notify Maintainer: @guidosarducci