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

PAM-based authentication ? #9

Open
asmodehn opened this issue Nov 3, 2020 · 1 comment
Open

PAM-based authentication ? #9

asmodehn opened this issue Nov 3, 2020 · 1 comment

Comments

@asmodehn
Copy link

asmodehn commented Nov 3, 2020

I had a look at the code while attempting to setup a tornado server with http auth based on PAM and this :

challenge = check_credentials_func(username)
        if not challenge:
            raise self.SendChallenge()

        if challenge == password:

makes it somewhat incompatible with PAM authentication, as far as I can tell...

For reference : https://github.com/FirefighterBlu3/python-pam/blob/master/pam.py provides an authenticate(username, password) function where one passes the user and the password at once, and the return value (after some waiting time) determine if we are logged in or not.

Any hint/ideas on how to improve this ?

@asmodehn
Copy link
Author

asmodehn commented Nov 4, 2020

see #10 for one attempt.
It seems that the current design is not ideal for this...

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

1 participant