Tesorio-specific changes #5
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
FabioFleitas
approved these changes
Aug 6, 2019
caioariede
added a commit
that referenced
this pull request
Aug 6, 2019
* Version 2.1.0, add DEFAULT_NEXT_URL * Add optional ASSERTION_URL setting We pass the result of `get_current_domain` to pysaml2 to validate the source of the incoming StatusResponse (the scheme+host of the request). This is not viable in a reverse proxy scenario, in particular when the host, port, and/or scheme of the django server changes with churn. As a solution, alter `get_current_domain` to just use a setting variable, `ASSERTION_URL`, instead of the scheme and host of the incoming SAML request. If the setting is not present, use the current behavior. * Pythonic map value check * Describe ASSERTION_URL in readme * Update AUTHORS.rst * release 2.1.1 * Import reverse from django.url for versions >= 2.0 * removed local temp file creation see https://pysaml2.readthedocs.io/en/latest/howto/config.html#metadata * update version 2.1.2 * Use Group.set() instead of assignment for Django 2.0 support * unquoting the logout view method reference * adding optional configuration for ADFS SAML compatibility * Merged PRs and did some cleaning * Allow pysaml2 to be a more recent version. * fix django 2 compatibility * Add name to AUTHORS as requested in docs, added djangorestframework-jwt dependency, made the User model come from get_user_model method * Added JWT authentication to views.py through use of a settings file. New settings: USE_JWT and FRONTEND_URL. Added django-rest-auth to setup.py * Better handling of accessing USE_JWT setting in views.py * Added is True check to USE_JWT setting call * Added support for local metadata file. * Added new user creation setting. * Update README.rst * fixed is_safe_url in django 2.1 * Tesorio-specific changes (#5)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.