chore: sync upstream changes from grafana-cold-storage/django-saml2-auth#2
Merged
Merged
Conversation
* Update deps * Add TRIGGER.EXTRACT_USER_IDENTITY hook function to extract_user_identity * Update README * Format code * Update tests to reflect changes in extract_user_identity function * Add new test for testing EXTRACT_USER_IDENTITY * Submit coverage and PyPI package only once * Create a release just once * Fix mypy errors * Revert "Submit coverage and PyPI package only once" * This reverts commit 7f7fb0f. * Update GitHub release action
…-cold-storage#345) * [FEAT] Implement a setting to disable the exception handler. * Some fixes and improvements
* feat: Allow custom assertion URIs * Update README * Remove CUSTOM_ASSERTION_URI * Lambda to func * URI to URL
* feat: Separate the Build and Deploy job * Change syntax
* feat: Trgger to get custom frontend URLs * Fix the typing * Try parallel coveralls
…URLS` and add an example (grafana-cold-storage#355)
* Update Django versions and poetry version * Update deps
… return the JWT (grafana-cold-storage#361) * Move use_jwt block below login block * Update test to verify the user is logged in to both the Single Page App and the Django admin console * Fix linting
* Update deps * Remove interrogate to fix CVE-2022-42969 * Update Django versions * Update poetry to v2 * Use the latest released version of Django * Replace poetry with uv
* Update uv.lock * Fix workflows * Use uv to run python build command
This reverts commit 345ab42.
* Update Django versions to check compatibility * Update action version and permissions * Fix permissions
…rafana-cold-storage#366) * Handle case where the frontend url already contains query parameters * Fix grafana-cold-storage#365 * Add test for new feature * Use JWT since tokens are only appended in the JWT scenario * Fix uv.lock * Parse query string and keep backward compatibility * Fix linter issues --------- Co-authored-by: Mostafa Moradian <mostafa@grafana.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
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.
This PR syncs
django-saml2-authwith upstreamgrafana-cold-storage/django-saml2-auth.mainmainIncluded upstream commit subjects:
TRIGGER.GET_METADATA_AUTO_CONF_URLSand add an example (Add clarity on the return object type of TRIGGER.GET_METADATA_AUTO_CONF_URLS grafana-cold-storage/django-saml2-auth#355)This is a straight upstream sync; no fork-specific changes were introduced.