Page MenuHomePhabricator

Centralize SUL2 and SUL3 device detection
Closed, ResolvedPublic

Description

SUL2 uses mobile=1 URL param and passes that around as a way to indicate the device that is making the request but SUL3 uses the useformat=mobile|desktop approach. With the introduction of the AuthPreserveQueryParams hook in https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1034966 and the way SUL3 currently uses the useformat param (with it being preserved by the hook), sync SUL2 to be compliant with SUL3 in the way they indicate the device making the request.

Event Timeline

DAlangi_WMF changed the task status from Open to In Progress.Nov 22 2024, 8:11 PM
DAlangi_WMF updated the task description. (Show Details)

Change #1094518 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@master] Make use of `useformat` query param for device detection

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

Change #1094518 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] Use `useformat` query param for device detection or mobile domain (m.)

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

Change #1098622 had a related patch set uploaded (by Gergő Tisza; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@wmf/1.44.0-wmf.4] Use `useformat` query param for device detection or mobile domain (m.)

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

Change #1098623 had a related patch set uploaded (by Gergő Tisza; author: Derick Alangi):

[mediawiki/extensions/CentralAuth@wmf/1.44.0-wmf.5] Use `useformat` query param for device detection or mobile domain (m.)

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

Change #1098633 had a related patch set uploaded (by Gergő Tisza; author: Gergő Tisza):

[operations/mediawiki-config@master] Fix mobile domain logic for login.wikimedia.org

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

Change #1098633 merged by jenkins-bot:

[operations/mediawiki-config@master] Fix mobile domain logic for login.wikimedia.org

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

Mentioned in SAL (#wikimedia-operations) [2024-11-27T22:51:16Z] <tgr@deploy2002> Started scap sync-world: Backport for [[gerrit:1098633|Fix mobile domain logic for login.wikimedia.org (T380646)]]

Mentioned in SAL (#wikimedia-operations) [2024-11-27T22:56:47Z] <tgr@deploy2002> tgr: Backport for [[gerrit:1098633|Fix mobile domain logic for login.wikimedia.org (T380646)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-11-27T23:09:24Z] <tgr@deploy2002> Finished scap sync-world: Backport for [[gerrit:1098633|Fix mobile domain logic for login.wikimedia.org (T380646)]] (duration: 18m 07s)

Change #1098623 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@wmf/1.44.0-wmf.5] Use `useformat` query param for device detection or mobile domain (m.)

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

Change #1098622 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@wmf/1.44.0-wmf.4] Use `useformat` query param for device detection or mobile domain (m.)

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

Mentioned in SAL (#wikimedia-operations) [2024-11-28T14:23:29Z] <urbanecm@deploy2002> Started scap sync-world: Backport for [[gerrit:1098623|Use useformat query param for device detection or mobile domain (m.) (T380646 T375788)]], [[gerrit:1098913|ReportIncident: Enable instrumentation on labs (T372823)]], [[gerrit:1098509|Enable message group subscription feature for some wikis (T372386)]], [[gerrit:1098622|Use useformat query param for device detection or mobile domain (m.) (

Mentioned in SAL (#wikimedia-operations) [2024-11-28T14:28:41Z] <urbanecm@deploy2002> urbanecm, tgr, abi, mszabo: Backport for [[gerrit:1098623|Use useformat query param for device detection or mobile domain (m.) (T380646 T375788)]], [[gerrit:1098913|ReportIncident: Enable instrumentation on labs (T372823)]], [[gerrit:1098509|Enable message group subscription feature for some wikis (T372386)]], [[gerrit:1098622|Use useformat query param for device detection or mobile domain (m.

Mentioned in SAL (#wikimedia-operations) [2024-11-28T14:54:04Z] <urbanecm@deploy2002> Finished scap sync-world: Backport for [[gerrit:1098623|Use useformat query param for device detection or mobile domain (m.) (T380646 T375788)]], [[gerrit:1098913|ReportIncident: Enable instrumentation on labs (T372823)]], [[gerrit:1098509|Enable message group subscription feature for some wikis (T372386)]], [[gerrit:1098622|Use useformat query param for device detection or mobile domain (m.)