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

Develop #40

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Develop #40

wants to merge 11 commits into from

Conversation

eldo-greshard
Copy link

when i implement this code to my project, it should have to have the same auth_user_mode on settings between client and server. so if my client using custom user auth it throw this.

Error :

django-simple-sso authorization
Manager isn't available; 'auth.User' has been swapped for 'users.User'

and the other problem is, when i tried to login to django admin it throw error that credential error with the same password, and lately i found out that the build user function set the password to unusable_password.

To avoid the user manager error for custom user who use abstract user.
prevent user password changed when build user
prever to let password as it is on the client side
settings.SSO_HOME_REDIRECT on LogoutClassView
settings.SSO_DASHBOARD_REDIRECT on LoginClassView
try to fix itsdangerous error when try to reach sso server addres.
fix type still temporary with no information to user, when itsdangerous can't get any token, it will redirect to sso client web.
should define SSO_CLIENT in settings file
@FinalAngel
Copy link
Member

@Karmael thank you very much and sorry for the late response. Would you be able to resolve the conflicts and add tests for it?

@eldo-greshard
Copy link
Author

@Karmael thank you very much and sorry for the late response. Would you be able to resolve the conflicts and add tests for it?

hey man, I'll write test for it and try to resolve the problem

@FinalAngel
Copy link
Member

That would be awesome @Karmael

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