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

Auth-Type is CHAP, passwords can be exposed in clear in freeradius logs #13

Closed
peppelinux opened this issue Nov 5, 2017 · 3 comments
Closed

Comments

@peppelinux
Copy link

peppelinux commented Nov 5, 2017

Hi Rob,
Is there any possibility to introduce an mschap auth type instead of a CHAP method as actually it does.
If a sysadmin configures a freeradius to log all the auth good_pass and bad_pass he could be also read all the password in clear, in /var/log/freeradius/radius.log.

The question could be also asked this way:
Can pyrad client stablish an auth session in mschap instead of chap?

@peppelinux
Copy link
Author

peppelinux commented Nov 7, 2017

The problem is in pyrad implementation, not django-radius. I also asked to pyrad author about this issue, as enanchment request as it actually is:

pyradius/pyrad#40

@peppelinux
Copy link
Author

peppelinux commented Nov 10, 2017

With the examples in pyprotosim I implemented a pure eapol_test python client program, downloadable here: https://github.com/peppelinux/pyEapol_test

I'll ask to pyrad author if there will be some possibility to introduce PEAP auth-type into pyrad, starting on this client or something similar. With a radius auth_backend that permits only PEAP auth-type we would be very strong on the security side.

@robgolding
Copy link
Owner

Hey @peppelinux! If you're able to get PEAP auth working with pyrad, then you could create a subclass of RADIUSBackend to use it. Unfortunately until then there's not much we can do on this end. I'm closing this issue now, but please feel free to open a PR in future once the upstream changes have been made. And thanks for the interesting in the project!

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

No branches or pull requests

2 participants