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 <yshve...@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=""



_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev

Reply via email to