[LEDE-DEV] [PATCH] Adding acct_interval to wireless configuration

[tapper]

Hi there mate thanks for your first patch!I am not a coder or  a dev. I 
just wanted to let you know that OpenWRT and LEDE are in the middle of 
re merging, so if you don't get any feedback for your patch it's because 
people are trying to sort out the re merging not ignoring you. If you 
dont get your patch merged or any feed back in 1-2 weeks then resend 
your mail. I am glad that we are getting new people around so thanks 
again. Tapper

On 29/05/2017 13:04, Yury Shvedov wrote:
> Hi, I'm new here and I wonder why there is still no UCI option to configure Radius-Accounting-Interim-Interval.
> I've made a patch to fix that and like to share it to others and ask is there any serious reason for lack of such
> option?
> 
> Make an ability to configure Accounting-Interim-Interval via UCI
> 
> Signed-off-by: Yury Shvedov <yshvedov at wimarksystems.com>
> ---
>   package/network/services/hostapd/files/hostapd.sh | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/package/network/services/hostapd/files/hostapd.sh b/package/network/services/hostapd/files/hostapd.sh
> index 6fb902e376..e6bccef39b 100644
> --- a/package/network/services/hostapd/files/hostapd.sh
> +++ b/package/network/services/hostapd/files/hostapd.sh
> @@ -151,6 +151,7 @@ hostapd_common_add_bss_config() {
>   	config_add_string acct_server
>   	config_add_string acct_secret
>   	config_add_int acct_port
> +	config_add_int acct_interval
>    	config_add_string dae_client
>   	config_add_string dae_secret
> @@ -208,7 +209,7 @@ hostapd_set_bss_options() {
>   		wps_independent wps_device_type wps_device_name wps_manufacturer wps_pin \
>   		macfilter ssid wmm uapsd hidden short_preamble rsn_preauth \
>   		iapp_interface eapol_version acct_server acct_secret acct_port \
> -		dynamic_vlan ieee80211w
> +		acct_interval dynamic_vlan ieee80211w
>    	set_default isolate 0
>   	set_default maxassoc 0
> @@ -249,6 +250,8 @@ hostapd_set_bss_options() {
>   		append bss_conf "acct_server_port=$acct_port" "$N"
>   		[ -n "$acct_secret" ] && \
>   			append bss_conf "acct_server_shared_secret=$acct_secret" "$N"
> +		[ -n "$acct_interval" ] && \
> +			append bss_conf "radius_acct_interim_interval=$acct_interval" "$N"
>   	}
>    	local vlan_possible=""
>