attachAccount.php uses the login attachment method (the $method parameter to CentralAuthUser::attach()). That's for newly autocreated accounts which start as attached, which is obviously not the case for accounts the script is used for. It should probably use admin or maybe a dedicated new method.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| attachAccount: Set attachment method to 'admin' | mediawiki/extensions/CentralAuth | master | +1 -1 |
Related Objects
Related Objects
- Mentioned In
- T378663: Add username parameter to attachAccount.php
- Mentioned Here
- T201477: Mention the status that the account is created due to imported version
T302771: Local accounts forcibly created with Special:CreateLocalAccount should have a unique method in Special:CentralAuth
T304015: Change CentralAuth database field localuser.lu_attached_method to not be an enum
Event Timeline
Comment Actions
Note:
(1) Currently lu_attached_method is enum and we need schema change to introduce new method.
(2) Also another field lu_attachment_method is added but currently unused (see T304015: Change CentralAuth database field localuser.lu_attached_method to not be an enum)
(3) "login" attachment method is currently (mis)used for other things, see T201477 and T302771.
In T304015#7808836 I propose to remove the "attachment method" concept completely.
Comment Actions
In previous weeks many Wikitech accounts are attached to SUL, and they all used "login" method.
Comment Actions
Change #1123328 had a related patch set uploaded (by Zabe; author: Zabe):
[mediawiki/extensions/CentralAuth@master] attachAccount: Set attachement method to 'admin'
Comment Actions
Change #1123328 merged by jenkins-bot:
[mediawiki/extensions/CentralAuth@master] attachAccount: Set attachment method to 'admin'