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

feat(oauth): Add support for configuring and passing hd parameter to Google #5162

Open
wants to merge 7 commits into
base: next
Choose a base branch
from

Conversation

notm1ke
Copy link

@notm1ke notm1ke commented Feb 17, 2025

This PR adds support for passing the hd parameter to Google's OAuth endpoint, which allows for a hint to appear in the login screen with the organization the login prompt is intended for. Some screenshots of this are provided below:

image

New field for tenant + helper message in the UI

image
What this actually does on the Google side

Changes

  • Added support for using the existing tenant column in the oauth_settings table to be passed forward as the hd parameter to the Google OAuth screen and display the hint.
  • Added a field in the OAuth config UI to configure the tenant value, a helper tooltip that explains what it does, and a link to the Google docs.
  • Added GoogleExtendSocialite composer dependency and config/services.php entry to match the convention used for Azure and Authentik
  • Validated login flow works as intended and passes the parameter only when it is configured

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.

2 participants