Skip to content

Conversation

@wxiaoguang
Copy link
Contributor

@wxiaoguang wxiaoguang commented Oct 25, 2025

Fix #35744 by reverting #18491

  • "OpenID" options don't mean "OAuth2Client" options
  • "OAuth2(server)" options don't mean "OAuth2Client" options

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 25, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Oct 25, 2025
@wxiaoguang wxiaoguang changed the title Revert #18491 Revert #18491, fix oauth2 client link account Oct 25, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Oct 25, 2025
@wxiaoguang
Copy link
Contributor Author

Doesn't block 1.25.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/v1.25 lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/go Pull requests that update Go code type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Oauth2 Provider must be enabled for Account Linking by login to work on Oauth2 Client

3 participants