Page MenuHomeVyOS Platform

Accel-ppp sending accounting interim updates acct-interim-interval option
Closed, ResolvedPublicFEATURE REQUEST

Description

accel-pppd supports sending accounting interim updates to RADIUS as indicated in the following document: accel-pppd

It indicates the following can be applied in the [radius] section:

acct-interim-interval=n By default is not defined.
Specifies the interval in seconds to send accounting information (may be overridden by radius Acct-Interim-Interval attribute)

This configuration setting is not supported in the VyOS CLI from what I can see, nor is the change persistent upon reboot when modifying the /run/accel-pppd/l2tp.conf file.

It would be great to introduce this configuration option so accounting information can be sent to RADIUS.

Details

Difficulty level
Unknown (require assessment)
Version
-
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Unspecified (possibly destroys the router)
Issue type
Improvement (missing useful functionality)

Related Objects

Mentioned In
1.3.5

Event Timeline

joshua.hanley renamed this task from Sending account interim upates to Sending accounting interim updates.May 9 2023, 8:16 AM
joshua.hanley updated the task description. (Show Details)
Viacheslav renamed this task from Sending accounting interim updates to Accel-ppp sending accounting interim updates acct-interim-interval option.May 9 2023, 9:28 AM
Viacheslav changed the task status from Open to In progress.
Viacheslav claimed this task.

PR https://github.com/vyos/vyos-1x/pull/1986

set service pppoe-server authentication mode 'radius'
set service pppoe-server authentication radius accounting-interim-interval '60'
set service pppoe-server authentication radius server 203.0.113.1 key '123'
set service pppoe-server client-ip-pool name POOL-01 gateway-address '192.0.2.1'
set service pppoe-server client-ip-pool name POOL-01 subnet '192.0.2.0/24'
set service pppoe-server interface eth1

@Viacheslav Thanks for the prompt response. Not sure if the change will also cover L2TP as well. For example:

set vpn l2tp remote-access authentication radius accounting-interim-interval '60'

@Viacheslav Thanks for the prompt response. Not sure if the change will also cover L2TP as well. For example:

set vpn l2tp remote-access authentication radius accounting-interim-interval '60'

This commit is for only PPPoE/IPoE/SSTP servers.
I'll create a separate PR for L2tp as it uses old-style python3 scripts.

Viacheslav changed the task status from In progress to Needs testing.May 11 2023, 6:45 AM
Viacheslav moved this task from Need Triage to Finished on the VyOS 1.4 Sagitta board.