(incubator-answer) branch fix/1.3.5/user created (now 389d8551)

[li...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

linkinstar pushed a change to branch fix/1.3.5/user
in repository https://gitbox.apache.org/repos/asf/incubator-answer.git


      at 389d8551 fix(user): remove user center log

This branch includes the following new commits:

     new 389d8551 fix(user): remove user center log

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

(incubator-answer) 01/01: fix(user): remove user center log

[li...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

linkinstar pushed a commit to branch fix/1.3.5/user
in repository https://gitbox.apache.org/repos/asf/incubator-answer.git

commit 389d8551f2dc804311ca83f8e86dfb4dd469a14d
Author: LinkinStars <li...@foxmail.com>
AuthorDate: Mon May 27 16:35:02 2024 +0800

    fix(user): remove user center log
---
 internal/repo/user/user_repo.go | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/internal/repo/user/user_repo.go b/internal/repo/user/user_repo.go
index 568ebf90..afe39410 100644
--- a/internal/repo/user/user_repo.go
+++ b/internal/repo/user/user_repo.go
@@ -291,10 +291,6 @@ func tryToDecorateUserInfoFromUserCenter(ctx context.Context, data *data.Data, o
 		return errors.BadRequest(reason.UserNotFound).WithError(err).WithStack()
 	}
 
-	// In general, usernames should be guaranteed unique by the User Center plugin, so there are no inconsistencies.
-	if original.Username != userCenterBasicUserInfo.Username {
-		log.Warnf("user %s username is inconsistent with user center", original.ID)
-	}
 	decorateByUserCenterUser(original, userCenterBasicUserInfo)
 	return nil
 }