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

Simplify Setup for a Django Project #12

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

Conversation

RevolutionTech
Copy link

This PR contains #11.

This PR also simplifies the setup and integration of django-saml-service-provider into a Django project by allowing settings in settings.py to be configured making the views in saml_service_provider.views usable without the user having to create their own views.

It also makes the provided auth backend more flexible and extensible, without having to override the entire authenticate() method for most use cases.

This PR also allows first and last names to be missing from the SAML response, and updates user attributes on every sign-in, instead of just when creating the user.

@nickatnight
Copy link

nickatnight commented Mar 20, 2018

There is no logout view?

@RevolutionTech
Copy link
Author

That's right, a logout view is not provided because django.contrib.auth.views.logout is sufficient.

@KevinHock
Copy link

Woah nice PRs @RevolutionTech, check out https://github.com/python-security/pyt sometime ;)

@fzipi
Copy link

fzipi commented May 16, 2018

👍 To merge this one.

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.

4 participants