Skip to content

Comments

Feat/update google client settings#2427

Merged
kevinchalet merged 4 commits intoopeniddict:devfrom
StellaAlexis:feat/update-google-client-settings
Feb 22, 2026
Merged

Feat/update google client settings#2427
kevinchalet merged 4 commits intoopeniddict:devfrom
StellaAlexis:feat/update-google-client-settings

Conversation

@StellaAlexis
Copy link
Contributor

Small PR to add the prompt value to the Google Web Integration configuration. Docs of the parameter can be found here
https://developers.google.com/identity/openid-connect/openid-connect#authenticationuriparameters

Added a new setting for the 'prompt' parameter in OpenIddictClientWebIntegrationProviders.xml to allow for user account selection and consent prompts.
@kevinchalet
Copy link
Member

Thanks for your PR!

FYI, adding the setting the .xml isn't enough (it won't do anything, basically). You also need to attach the prompt value to the request. See how it's done for the other providers:

// By default, Zoho doesn't return a refresh token but
// allows sending an "access_type" parameter to retrieve one.
else if (context.Registration.ProviderType is ProviderTypes.Zoho)
{
var settings = context.Registration.GetZohoSettings();
context.Request["access_type"] = settings.AccessType;
context.Request.Prompt = settings.Prompt;
}

@StellaAlexis
Copy link
Contributor Author

Ah thanks for pointing that out! I think I added the required extra change now. Also checked if there was anything else that needed updating or any tests around this part but I couldn't find any.

Copy link
Member

@kevinchalet kevinchalet left a comment

Choose a reason for hiding this comment

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

Thanks! ❤️

@kevinchalet kevinchalet merged commit ef0092a into openiddict:dev Feb 22, 2026
6 checks passed
@kevinchalet
Copy link
Member

Merged. Congrats for your first contribution to OpenIddict! 🎉

@kevinchalet kevinchalet added this to the 7.3.0 milestone Feb 22, 2026
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.

2 participants