From fe74774d3203a66e7e18fe6a0ffdfb530bbc624b Mon Sep 17 00:00:00 2001 From: Hunter Morgan Date: Mon, 19 May 2025 20:29:24 -0400 Subject: [PATCH] enhance comment --- connector/oidc/oidc.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/connector/oidc/oidc.go b/connector/oidc/oidc.go index 29548c09..d0790996 100644 --- a/connector/oidc/oidc.go +++ b/connector/oidc/oidc.go @@ -358,7 +358,7 @@ func (c *oidcConnector) LoginURL(s connector.Scopes, callbackURL, state string) if s.OfflineAccess { opts = append(opts, oauth2.AccessTypeOffline) - // Only add prompt parameter if it's not "none" + // `promptType: "none"` is used in scenarios where user interaction is not possible or desirable, such as automated workflows or CLI tools. However, the conditional ensures that the parameter is omitted when it is not supported by the identity provider to prevent errors (like OneLogin). if c.promptType != "none" { opts = append(opts, oauth2.SetAuthURLParam("prompt", c.promptType)) }