Last modified: 2012-01-14 01:12:57 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T25802, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23802 - Let AuthPlugin decide whether a specific user (name) can be auto created
Let AuthPlugin decide whether a specific user (name) can be auto created
Status: NEW
Product: MediaWiki
Classification: Unclassified
User login and signup (Other open bugs)
1.17.x
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-reviewed
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-05 03:08 UTC by Liangent
Modified: 2012-01-14 01:12 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Add a parameter $user to autoCreate (897 bytes, patch)
2010-06-05 03:08 UTC, Liangent
Details

Description Liangent 2010-06-05 03:08:27 UTC
Created attachment 7430 [details]
Add a parameter $user to autoCreate

See the patch. This will not break any existing code.
Comment 1 Platonides 2010-11-10 23:58:31 UTC
I don't feel this is a good place to do it.

The use case itself is a bit convoluted. The user name cannot be autocreated, but it can login if the global account exists. I think it would be easier to deny that user attaching global account. I don't think AuthPlugin::autoCreate() is the appropiate place for checking per-wiki autocreation policies.
Comment 2 Brion Vibber 2011-04-05 01:37:38 UTC
I've removed this as dep for other bug since I don't think it actually helps the case where _another_ plugin should be making the determination.

For the case where the auth plugin itself wants more control, possibly the newer external authentication interface provides more of that, but haven't had a chance to look it over.
Comment 3 bsitu 2012-01-14 01:12:57 UTC
Marked as reviewed per comment 1 and 2

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links