netifd: support DHCP sendopts as list options
authorHans Dedecker <dedeckeh@gmail.com>
Sat, 24 Feb 2018 15:52:38 +0000 (16:52 +0100)
committerHans Dedecker <dedeckeh@gmail.com>
Tue, 27 Feb 2018 12:36:46 +0000 (13:36 +0100)
Support config in the form of ....
add_list sendopts=router:10.10.10.2
add_list sendopts=nissrv:20.20.20.2
add_list sendopts=0x7D:abba

This allows to configure sendopts having white spaces as option value

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
package/network/config/netifd/Makefile
package/network/config/netifd/files/lib/netifd/proto/dhcp.sh

index d2210e7c2e148917fca62180c3a66e002b9a2f7c..d62b60ad98941cddb29d56c64db258fbb660e445 100644 (file)
@@ -1,7 +1,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=netifd
-PKG_RELEASE:=3
+PKG_RELEASE:=4
 
 PKG_SOURCE_PROTO:=git
 PKG_SOURCE_URL=$(PROJECT_GIT)/project/netifd.git
index 385cfc397474f94eee37df500dd7c35a5ecef6a3..afcfba9f49a737dca9f75ca25aa0ad59a60cd171 100755 (executable)
@@ -16,7 +16,7 @@ proto_dhcp_init_config() {
        proto_config_add_string 'reqopts:list(string)'
        proto_config_add_boolean 'defaultreqopts:bool'
        proto_config_add_string iface6rd
-       proto_config_add_string sendopts
+       proto_config_add_array 'sendopts:list(string)'
        proto_config_add_boolean delegate
        proto_config_add_string zone6rd
        proto_config_add_string zone
@@ -25,21 +25,23 @@ proto_dhcp_init_config() {
        proto_config_add_boolean classlessroute
 }
 
+proto_dhcp_add_sendopts() {
+       [ -n "$1" ] && append "$3" "-x $1"
+}
+
 proto_dhcp_setup() {
        local config="$1"
        local iface="$2"
 
        local ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd sendopts delegate zone6rd zone mtu6rd customroutes classlessroute
-       json_get_vars ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd sendopts delegate zone6rd zone mtu6rd customroutes classlessroute
+       json_get_vars ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd delegate zone6rd zone mtu6rd customroutes classlessroute
 
        local opt dhcpopts
        for opt in $reqopts; do
                append dhcpopts "-O $opt"
        done
 
-       for opt in $sendopts; do
-               append dhcpopts "-x $opt"
-       done
+       json_for_each_item proto_dhcp_add_sendopts sendopts dhcpopts
 
        [ -z "$hostname" ] && hostname="$(cat /proc/sys/kernel/hostname)"
        [ "$defaultreqopts" = 0 ] && defaultreqopts="-o" || defaultreqopts=