Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

server: fix listen for CoAs without auth, neither acct , add runtime CoA+disconnect example #59

Merged
merged 2 commits into from
Jul 5, 2018

Conversation

vjardin
Copy link
Contributor

@vjardin vjardin commented Aug 15, 2017

When auth_enabled is False and acct_enabled is False, we cannot start
a listener for CoA only.

Fixes: db780ca ("add coa server support")
Fixes: 40c7eab ("historics")

Vincent JARDIN added 2 commits August 15, 2017 21:39
When auth_enabled is False and acct_enabled is False, we cannot start
a listener for CoA only.

Fixes: db780ca ("add coa server support")
Fixes: 40c7eab ("historics")
The coa.py example needed to be modified for every tests of CoA, Disconnect and
when a NAS-Identifier needs to be inserted.

Example of use of this new feature:

 # start the RADIUS server
pyrad/example$ ./server.py &

 # send CoA to the RADIUS server
pyrad/example$ ./coa.py coa fooId-1234
 [server] Received an coa request
 [server] Attributes:
 [server] NAS-Identifier: [u'fooId-1234']
 [server] Acct-Session-Id: [u'1337']

 [coa] {}
 [coa] 44

 # send Disconnect to the RADIUS server
pyrad/example$ ./coa.py dis fooId-1234
 [server] Received an disconnect request
 [server] Attributes:
 [server] NAS-Identifier: [u'fooId-1234']
 [server] Acct-Session-Id: [u'1337']

 [coa] {}
 [coa] 45
@vjardin vjardin changed the title server: listen for CoAs without auth, neither acct server: fix listen for CoAs without auth, neither acct , add runtime CoA+disconnect example Aug 15, 2017
@GIC-de GIC-de self-requested a review July 5, 2018 20:09
@GIC-de GIC-de self-assigned this Jul 5, 2018
@GIC-de GIC-de merged commit 3b44c71 into pyradius:master Jul 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants