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

2017-06-27 Thread Stijn Tintel
On 27-06-17 09:34, yshve...@wimarksystems.com wrote: > From: Yury Shvedov > > Sorry, I've tried to configure my thunderbird properly to use it as > patch sender. But it seems unsuccessfully. Do I understand you > correctly about grouping acct_* values? > No worries, it's OK now. Added to my stagin

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

2017-06-27 Thread yshvedov
From: Yury Shvedov Sorry, I've tried to configure my thunderbird properly to use it as patch sender. But it seems unsuccessfully. Do I understand you correctly about grouping acct_* values? Make an ability to configure Accounting-Interim-Interval via UCI Signed-off-by: Yury Shvedov --- packag

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

2017-06-26 Thread Stijn Tintel
On 27-06-17 08:23, Yury Shvedov wrote: > Like that? No, the patch is still mangled, and it is not sent with git send-email, but with Thunderbird. Please have a look at https://www.kernel.org/pub/software/scm/git/docs/git-send-email.html, you can pass your SMTP server credentials to it. Alternativel

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

2017-06-26 Thread Yury Shvedov
Like that? Make an ability to configure Accounting-Interim-Interval via UCI Signed-off-by: Yury Shvedov --- package/network/services/hostapd/files/hostapd.sh | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/package/network/services/hostapd/files/hostapd.sh b/package/

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

2017-06-26 Thread 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 a

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

2017-05-30 Thread Yury Shvedov
Hi, Nick! This is good point, but radius acct could be configured without radius auth. Even for open, wep and wpa/psk networks. I'm using radius acct to collect wireless client statistic among all of my networks. So there no Access-Accept messages available. On 05/30/2017 11:42 PM, Nick Low

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

2017-05-30 Thread Nick Lowe
Hi Yury, Have you tried returning an Acct-Interim-Interval attribute in the Access-Accept packets sent by your RADIUS server, which should configure this on a per client/station basis? Cheers, Nick ___ Lede-dev mailing list Lede-dev@lists.infradead.or

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

2017-05-29 Thread 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

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

2017-05-29 Thread Yury Shvedov
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