Page MenuHomePhabricator
Create Task
Maniphest T73064

Quarry should update username when a user is renamed
Closed, ResolvedPublicBUG REPORT
Actions

Description

The first time I logged in to Quarry was before my global account was renamed.

Now, if I go to
http://quarry.wmflabs.org/
and click in the login button, it shows

Hi He7d3r,

SQL Quarry would like to have basic
access on your behalf on
meta.wikimedia.org.

(which uses my current username).
However, after I click in the button to allow the login, it shows my old username in the top right corner, with a link using the old username:
http://quarry.wmflabs.org/Helder.wiki
The correct link
http://quarry.wmflabs.org/He7d3r
returns "502 Bad Gateway"

How can I force it to use the new username?

Details

Reference
bz71064
SubjectRepoBranchLines +/-
Update username in database when a renamed user loginsanalytics/quarry/webmaster+13 -1
Customize query in gerrit

Related Objects

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:57 AM
• bzimport added a project: Quarry.
• bzimport set Reference to bz71064.
• bzimport added a subscriber: Unknown Object (MLST).
He7d3r created this task.Sep 19 2014, 7:55 PM
He7d3r updated the task description. (Show Details)Jan 3 2015, 3:19 PM
He7d3r set Security to None.
He7d3r mentioned this in T94333: Thanks log doesn't reflect user name change.Mar 29 2015, 7:19 PM
He7d3r mentioned this in T27377: Update username in AbuseFilter interface when user is renamed using Renameuser.
yuvipanda moved this task from Backlog to Bugs on the Quarry board.Jul 4 2015, 5:13 PM
yuvipanda added a comment.Jul 5 2015, 12:11 PM

Hmm, good question. I'm not particularly sure how to do this...

He7d3r added a comment.Jul 5 2015, 2:12 PM

I don't know if it is related but, I have a similar problem with the Wikidata Game, which also uses OAuth:
https://bitbucket.org/magnusmanske/wikidata-game/issue/153/stats-are-lost-after-a-global-account

yuvipanda added a comment.Jul 5 2015, 3:18 PM

It's probably a similar issue - we just cache the name OAuth gives us in the db and do not change it every login. We perhaps should...

Reedy removed a subscriber: • wikibugs-l-list.Dec 10 2015, 11:00 PM
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 10 2015, 11:00 PM
Ricordisamoa added a subscriber: Ricordisamoa.Dec 18 2015, 9:58 AM
Dalba awarded a token.Jan 7 2016, 9:23 AM
Dalba added a subscriber: Dalba.
zhuyifei1999 added a subscriber: zhuyifei1999.Jan 27 2016, 9:01 AM
Krenair added subscribers: Legoktm, Krenair.Feb 5 2016, 4:49 AM
In T73064#1427389, @yuvipanda wrote:

It's probably a similar issue - we just cache the name OAuth gives us in the db and do not change it every login. We perhaps should...

@yuvipanda: So I thought about doing that. I spent a while setting stuff up in beta, filing bugs for the various issues I encountered along the way, etc., and then I wrote a quick patch. But just before I went to test the patch, I realised this might happen:

  • A and B exist as users in Quarry.
  • A renames to C
  • B renames to A
  • B logs into Quarry (C, previously known as A, doesn't)
  • ... now we have two As? The username field is supposed to be unique according to tables.sql.

@Legoktm: That's a possible situation, right? Or do we disallow renaming to usernames that were taken in the past?

Stefan2 added a subscriber: Stefan2.Apr 25 2016, 6:47 PM
In T73064#2001109, @Krenair wrote:
  • ... now we have two As? The username field is supposed to be unique according to tables.sql.

This may also happen even if you do not implement user renames in Quarry. Consider the following situation:

  • A and B both have SUL accounts. A has a Quarry account, but B has no Quarry account.
  • A renames to C. C (previously A) remains Quarry user A because Quarry doesn't rename users.
  • B renames to A.
  • A (previously B) logs in to Quarry for the first time. Since A (previously B) hasn't logged in before, Quarry creates an account for A (previously B) under his current user name, A.

Now there are two "User:A" on Quarry.

Base added a subscriber: Base.May 6 2016, 1:37 PM

I do not know hot it is build so do not know if that might occur (well hopefully it is based on IDs and not on usernames behind the scenes so it might not), but I hope that those renames if you made them will not just create a second account for new username of a user and thus leave the first one orphaned. As it would be unfortunate to stack orphaned accounts.

• Phabricator_maintenance removed a subscriber: yuvipanda.Jun 7 2017, 6:57 PM
He7d3r renamed this task from Quarry shows old username after user was renamed to Quarry should update username when a user is renamed.Dec 17 2017, 12:44 PM
gerritbot added a subscriber: gerritbot.Dec 17 2017, 7:50 PM

Change 398711 had a related patch set uploaded (by Zhuyifei1999; owner: Zhuyifei1999):
[analytics/quarry/web@master] Update username in database when a renamed user logins.

https://gerrit.wikimedia.org/r/398711

gerritbot added a project: Patch-For-Review.Dec 17 2017, 7:50 PM
gerritbot added a comment.Sep 17 2018, 5:24 PM

Change 398711 merged by jenkins-bot:
[analytics/quarry/web@master] Update username in database when a renamed user logins

https://gerrit.wikimedia.org/r/398711

Framawiki closed this task as Resolved.Sep 17 2018, 5:39 PM
Framawiki assigned this task to zhuyifei1999.
Framawiki added a subscriber: Framawiki.

Tested after the merge of the above patch by

  1. Creating a test account on WM and log in into quarry
  2. Rename this account
  3. Confirming that, still logged in into quarry, the old username is still shown in the top right corner of the site
  4. By disconnect and log in again, new user is shown
  5. The new userpage is correctly rendered, but the old says "User not found".

It's not ideal, but it seems enough to me to close this task.

If you couldn't connect because of a renaming, you should be able to do so now.

Stashbot added a subscriber: Stashbot.Sep 17 2018, 5:41 PM

Mentioned in SAL (#wikimedia-cloud) [2018-09-17T17:40:59Z] <framawiki> deployed e8e6e02 to quarry-web-01 T73064

Aklapper changed the subtype of this task from "Task" to "Bug Report".Jul 18 2023, 12:20 PM
Maintenance_bot removed a project: Patch-For-Review.Jul 18 2023, 12:32 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL