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

Allow config section in python config #1469

Merged
merged 1 commit into from Jan 15, 2016

Conversation

herwinw
Copy link
Contributor

@herwinw herwinw commented Dec 22, 2015

They will recursively be turned into a dict, and added to radiusd.config. Behaviour is identical to the same config option in rlm_perl.

This is the extended version of #1451/#1453. It has not been addressed by #1459.

This patch can be backported to v3.0.x, it just adds another attribute to the radiusd module, script that don't use it just keep not using it ;)

They will recursively be turned into a dict, and added to radiusd.config. Behaviour is identical to the same config option in rlm_perl.

This is the extended version of FreeRADIUS#1451/FreeRADIUS#1453. It has *not* been addressed by FreeRADIUS#1459.
@herwinw
Copy link
Contributor Author

herwinw commented Dec 29, 2015

@alejandro-perez: You are the one who originally requested this feature, care to try/test it?

@alejandro-perez
Copy link
Contributor

Sure. Will try it in a few days, though. I'm on holidays now :)

@alejandro-perez
Copy link
Contributor

It works perfectly fine for the tests I've done. Thanks! (the most difficult part was figuring out that it was a different repository the one I had to clone :))

@alejandro-perez
Copy link
Contributor

Please, accept this request.

alandekok added a commit that referenced this pull request Jan 15, 2016
Allow config section in python config
@alandekok alandekok merged commit 4978716 into FreeRADIUS:v3.1.x Jan 15, 2016
@herwinw herwinw deleted the rlm_python_conf branch March 23, 2016 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants