From 0b1b1430be7025ab7944de100ba122e7cfae41fe Mon Sep 17 00:00:00 2001 From: lcgkm Date: Thu, 6 Nov 2025 11:08:21 +0800 Subject: [PATCH] Fix oauth2 sync error We'd better to use id to find a user. Not LoginName. And LoginName!= ExternalID --- services/auth/source/oauth2/source_sync.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/auth/source/oauth2/source_sync.go b/services/auth/source/oauth2/source_sync.go index c2e3dfb1a8ada..ac7ebaa04b7a3 100644 --- a/services/auth/source/oauth2/source_sync.go +++ b/services/auth/source/oauth2/source_sync.go @@ -47,7 +47,7 @@ func (source *Source) Sync(ctx context.Context, updateExisting bool) error { } func (source *Source) refresh(ctx context.Context, provider goth.Provider, u *user_model.ExternalLoginUser) error { - log.Trace("Syncing login_source_id=%d external_id=%s expiration=%s", u.LoginSourceID, u.ExternalID, u.ExpiresAt) + log.Trace("Syncing login_source_id=%d external_id=%s user_id=%s expiration=%s", u.LoginSourceID, u.ExternalID, u.UserID, u.ExpiresAt) shouldDisable := false @@ -62,7 +62,7 @@ func (source *Source) refresh(ctx context.Context, provider goth.Provider, u *us } user := &user_model.User{ - LoginName: u.ExternalID, + ID: u.UserID, LoginType: auth.OAuth2, LoginSource: u.LoginSourceID, }