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

[Stijn Tintel]

On 29-05-17 14: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

The patch is broken. Please use git send-email or create a PR on Github.

>
> 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
Please group the 4 acct_* directives on the same line, and combine the
resulting 2 short lines.
>   	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=""
Thanks,
Stijn