Can not login to gerrit!

[vista...@gmail.com]

Hi,all:

     

       auth.type = http  and a new account  which was create by restapi 'PUT /accounts/{username}'

       when login to gerrit,I got the error "com.google.gerrit.server.account.AccountUserNameException: Cannot assign user name "xxx" to account 101; name already in use." 

       when refresh the login web,new error occur."com.google.gerrit.server.account.AccountUserNameException: Cannot assign user name "xxx" to account 102; name already in use."

Why?

Thanks!

[Sven Selberg]

Which version of Gerrit?

Could it be this bug fixed in 2.14.7?

https://bugs.chromium.org/p/gerrit/issues/detail?id=7652

/Sven

[vista...@gmail.com]

2.13.7

[David Hayes]

Saw this issue in the past when swapping from LDAP auth to another type. The account was duplicated in the DB and had to be manually corrected.

[vista...@gmail.com]

Hi,David:

    The miss entry in  DB had to be manually corrected is too much trouble.

    And the content like change [1] should be merged to 2.13.x

Thanks!

 [1] https://gerrit-review.googlesource.com/c/gerrit/+/162450

[vista...@gmail.com]

--- a/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountManager.java

+++ b/gerrit-server/src/main/java/com/google/gerrit/server/account/AccountManager.java

@@ -127,6 +127,15 @@ public class AccountManager {

         AccountExternalId.Key key = id(who);

         AccountExternalId id = getAccountExternalId(db, key);

         if (id == null) {

+          if (who.getUserName() != null) {

+            Account.Id ext = lookup("username:" + who.getUserName());

+            if (ext != null){

+              // An inconsistency is detected in the database, having a record for scheme "username:"

+              // but no record for scheme "gerrit:". Try to recover by linking

+              // "gerrit:" identity to the existing account.

+              return link(ext, who);

+            }

+          }

           // New account, automatically create and return.

           //

           return create(db, who);

@@ -159,10 +168,10 @@ public class AccountManager {

           }

         }

       }

-      return null;

     }

     // We don't have at the moment an account_by_external_id cache

+    // or for whatever reason the account external id was not indexed

     // but by using the accounts cache we get the list of external_ids

     // without having to query the DB every time

     if (key.getScheme().equals(AccountExternalId.SCHEME_GERRIT)

[Edwin Kempin]

Sorry, but we don't accept patches through the mailing list.

Please push any contribution/patch to the Gerrit project (https://gerrit-review.googlesource.com/admin/repos/gerrit).

--
--
To unsubscribe, email repo-discuss...@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[thomasmu...@yahoo.com]

At the wmf we had this problem, @luca created this patch (was on the master branch but it was near sync with the 2.13 branch)

See https://gerrit-review.googlesource.com/c/gerrit/+/92830

On Friday, June 22, 2018 at 2:47:50 AM UTC+1, vista...@gmail.com wrote:

[thomasmu...@yahoo.com]

We doin't use that change anymore, now that we are on 2.15.

On Friday, June 22, 2018 at 2:47:50 AM UTC+1, vista...@gmail.com wrote: