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

rebase aplus 2406.4 again #90

Closed
wants to merge 65 commits into from
Closed

Conversation

m4ra
Copy link
Contributor

@m4ra m4ra commented Jun 28, 2024

Tasks

  • PR name contains story or task reference
  • Documentation (docs and inline)
  • Tests (including n+1 and django_assert_num_queries where applicable)
  • Changelog

Phillippa and others added 30 commits June 19, 2024 14:23
It allows to set a username if we only got an email from the saml2
IDP.
 - only show login link in header (to discourage registrations)
 - add saml2 login at the top of login page. Disabled if no saml2
config can be found
When set, the login directly forwards to the specified IDP, in
contrast to showing a full list. This does not prevent other IPDs
to send valid login tokens - thus the user needs to make sure the
own service is only registered on the desired IDP.
djangosaml2 does not offer a way to not login the user before doing
extra validation steps. In order to force users to sign up before
being able to use the website, add a simple middleware, redirecting
them to the signup as long as their email is not verified.

Logout and wagtail pages are excepted from these rules.
* email addresses are created by pysaml, use get_or_create() instead of
  create()
goapunk and others added 24 commits June 19, 2024 14:23
…during saml signup to allow aborting the signup

fixes #6
the users preferred language.

- add missing cookie settings to language cookie

fixes #2719
djangosaml2 does not offer a way to not login the user before doing
extra validation steps. In order to force users to sign up before
being able to use the website, add a simple middleware, redirecting
them to the signup as long as their email is not verified.

Logout and wagtail pages are excepted from these rules.
@m4ra m4ra changed the title Mk 2024 06 rebase aplus 2406.4 again rebase aplus 2406.4 again Jun 28, 2024
@m4ra m4ra requested a review from goapunk June 28, 2024 10:57
@m4ra m4ra force-pushed the mk-2024-06-rebase-aplus-2406.4-again branch from fd46869 to ee9f193 Compare June 28, 2024 11:06
@m4ra m4ra closed this Jun 28, 2024
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.

6 participants