In T344276, we want to make sure the SpecialPage::requireLogin() function is used in accordance with the introduction of temporary users. As for now, we want to treat temporary users just like anonymous or IP users.
Request:
Should requireLogin() be available to temporary users?
If yes, no update is needed.
If not, requireNamedUser must return true instead.
Notes:
RequireLogin was found in the following files:
includes/Special/SpecialContentTranslation.php
Code Search Link
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Restricted Task | |||||
| Resolved | kostajh | T294511 2021 Security Team wikireplicas audit | |||
| Declined | None | T284948 Raw IPs of logged-out users disclosed in wiki-replicas | |||
| In Progress | Niharika | T324492 Temporary accounts - MVP | |||
| Open | None | T326816 [Epic] Update features for temporary accounts | |||
| Resolved | Nikerabbit | T326879 Update Language Team-owned products that may be affected by IP Masking | |||
| Resolved | Nikerabbit | T344151 Review ContentTranslation for IP Masking | |||
| Resolved | kostajh | T344276 Should SpecialPages that return true for ::requireLogin allow access to temporary users? | |||
| Resolved | ngkountas | T344771 Update requireLogin() for temp users on ContentTranslation extension |
Change 965464 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):
[mediawiki/extensions/ContentTranslation@master] CX: Use "requireNamedUser" instead of "requireLogin"
Change 965464 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] CX Special Page: Use "requireNamedUser" instead of "requireLogin"
This task doesn't include any UI change and cannot be verified with screencasts, etc. However, the fix has been deployed for about a month now, and things continue to work as expected. This task can be closed as done.