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

[Google] Fix regression in v16 of no longer stripping username's domain if hosted_domain has a single entry #661

Merged

Conversation

consideRatio
Copy link
Member

@consideRatio consideRatio commented Aug 11, 2023

This PR influences users with GoogleOAuthenticator that had configured hosted_domain with a single domain entry.

This PR is fixing a regression / undocumented breaking change introduced in oauthenticator 16.0.0 by returning things as they were before.

@consideRatio consideRatio changed the title fix, google: don't include domain if hosted_domain has a single entry [Google] Fix regression in 16 no longer stripping domain if hosted_domain has a single entry Aug 11, 2023
@consideRatio consideRatio changed the title [Google] Fix regression in 16 no longer stripping domain if hosted_domain has a single entry [Google] Fix regression in 16 no longer stripping domain if hosted_domain has a single entry Aug 11, 2023
@consideRatio consideRatio changed the title [Google] Fix regression in 16 no longer stripping domain if hosted_domain has a single entry [Google] Fix regression in v16 of no longer stripping domain if hosted_domain has a single entry Aug 11, 2023
@consideRatio consideRatio changed the title [Google] Fix regression in v16 of no longer stripping domain if hosted_domain has a single entry [Google] Fix regression in v16 of no longer stripping username's domain if hosted_domain has a single entry Aug 11, 2023
Copy link
Member

@minrk minrk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Added one clarifying detail on what is stripped to the help string (already present elsewhere)

oauthenticator/google.py Outdated Show resolved Hide resolved
@consideRatio consideRatio merged commit 2c65356 into jupyterhub:main Aug 11, 2023
8 checks passed
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.

google auth in oauthenticator 16.x changes username format
2 participants