Page MenuHomePhabricator
Create Task
Maniphest T362700

GlobalBlockLogFormatter does not handle pre-2010 GlobalBlocking logs correctly, causing an InvalidArgumentException and missing log parameter data
Closed, ResolvedPublic2 Estimated Story PointsPRODUCTION ERROR
Actions

Description

Error
message
Invalid username: Contributions/67.159.44.24
trace
from /srv/mediawiki/php-1.42.0-wmf.26/extensions/CentralAuth/includes/User/CentralAuthUser.php(254)
#0 /srv/mediawiki/php-1.42.0-wmf.26/extensions/CentralAuth/includes/User/CentralAuthUser.php(231): MediaWiki\Extension\CentralAuth\User\CentralAuthUser::getInstanceByName(string)
#1 /srv/mediawiki/php-1.42.0-wmf.26/extensions/GlobalBlocking/includes/GlobalBlockLogFormatter.php(212): MediaWiki\Extension\CentralAuth\User\CentralAuthUser::getInstance(MediaWiki\User\UserIdentityValue)
#2 /srv/mediawiki/php-1.42.0-wmf.26/extensions/GlobalBlocking/includes/GlobalBlockLogFormatter.php(133): MediaWiki\Extension\GlobalBlocking\GlobalBlockLogFormatter->checkAuthorityCanSeeUser(MediaWiki\User\UserIdentityValue)
#3 /srv/mediawiki/php-1.42.0-wmf.26/includes/logging/LogFormatter.php(562): MediaWiki\Extension\GlobalBlocking\GlobalBlockLogFormatter->getMessageParameters()
#4 /srv/mediawiki/php-1.42.0-wmf.26/includes/logging/LogFormatter.php(537): LogFormatter->getActionMessage()
#5 /srv/mediawiki/php-1.42.0-wmf.26/includes/logging/LogEventsList.php(332): LogFormatter->getActionText()
#6 /srv/mediawiki/php-1.42.0-wmf.26/includes/logging/LogPager.php(452): LogEventsList->logLine(stdClass)
#7 /srv/mediawiki/php-1.42.0-wmf.26/includes/pager/ReverseChronologicalPager.php(134): MediaWiki\Pager\LogPager->formatRow(stdClass)
#8 /srv/mediawiki/php-1.42.0-wmf.26/includes/pager/IndexPager.php(594): MediaWiki\Pager\ReverseChronologicalPager->getRow(stdClass)
#9 /srv/mediawiki/php-1.42.0-wmf.26/includes/specials/SpecialLog.php(313): MediaWiki\Pager\IndexPager->getBody()
#10 /srv/mediawiki/php-1.42.0-wmf.26/includes/specials/SpecialLog.php(192): MediaWiki\Specials\SpecialLog->show(MediaWiki\Html\FormOptions, array)
#11 /srv/mediawiki/php-1.42.0-wmf.26/includes/specialpage/SpecialPage.php(718): MediaWiki\Specials\SpecialLog->execute(NULL)
#12 /srv/mediawiki/php-1.42.0-wmf.26/includes/specialpage/SpecialPageFactory.php(1669): MediaWiki\SpecialPage\SpecialPage->run(NULL)
#13 /srv/mediawiki/php-1.42.0-wmf.26/includes/actions/ActionEntryPoint.php(504): MediaWiki\SpecialPage\SpecialPageFactory->executePath(string, MediaWiki\Context\RequestContext)
#14 /srv/mediawiki/php-1.42.0-wmf.26/includes/actions/ActionEntryPoint.php(145): MediaWiki\Actions\ActionEntryPoint->performRequest()
#15 /srv/mediawiki/php-1.42.0-wmf.26/includes/MediaWikiEntryPoint.php(199): MediaWiki\Actions\ActionEntryPoint->execute()
#16 /srv/mediawiki/php-1.42.0-wmf.26/index.php(58): MediaWiki\MediaWikiEntryPoint->run()
#17 /srv/mediawiki/w/index.php(3): require(string)
#18 {main}
Impact

Causes Special:Log on commonswiki when filtered for GlobalBlocking log entries to throw an exception.

Notes

Looks to be caused rows such as:

+---------+----------+------------+----------------+---------------+----------------------------+----------------+------------+-------------+-----------+----------+
| log_id  | log_type | log_action | log_timestamp  | log_namespace | log_title                  | log_comment_id | log_params | log_deleted | log_actor | log_page |
+---------+----------+------------+----------------+---------------+----------------------------+----------------+------------+-------------+-----------+----------+
| 5038622 | gblblock | whitelist  | 20080827225222 |            -1 | Contributions/67.159.44.24 |       91105968 |            |           0 |    455357 |     NULL |
+---------+----------+------------+----------------+---------------+----------------------------+----------------+------------+-------------+-----------+----------+

Details

Request URL
https://commons.wikimedia.org/wiki/Special:Log?type=gblblock
SubjectRepoBranchLines +/-
Handle pre-2010 logs for IP rangesmediawiki/extensions/GlobalBlockingREL1_42+15 -2
Handle pre-2010 logs for IP rangesmediawiki/extensions/GlobalBlockingmaster+15 -2
Handle pre-2010 gblblock log entriesmediawiki/extensions/GlobalBlockingREL1_42+60 -6
Handle pre-2010 gblblock log entriesmediawiki/extensions/GlobalBlockingmaster+60 -6
Customize query in gerrit

Related Objects

Event Timeline

Dreamy_Jazz created this task.Tue, Apr 16, 5:02 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptTue, Apr 16, 5:02 PM
Dreamy_Jazz updated the task description. (Show Details)Tue, Apr 16, 5:03 PM
Dreamy_Jazz added a project: Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)).
Dreamy_Jazz set the point value for this task to 2.Tue, Apr 16, 5:07 PM
Dreamy_Jazz moved this task from Priority Backlog to In Progress on the Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)) board.
Dreamy_Jazz added a comment.Tue, Apr 16, 5:11 PM

There are also entries on enwiki and on metawiki. On metawiki there are 755 of these entries.

Dreamy_Jazz renamed this task from Invalid username: Contributions/67.159.44.24 to Target of old GlobalBlocking log entries is the contributions page of the IP target, causing a InvalidArgumentException when CentralAuth is installed.Tue, Apr 16, 5:12 PM
gerritbot added a comment.Tue, Apr 16, 5:36 PM

Change #1020297 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@master] Handle gblblock with their target as Special:Contributions

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

gerritbot added a project: Patch-For-Review.Tue, Apr 16, 5:36 PM
Dreamy_Jazz added a comment.Tue, Apr 16, 5:42 PM

Furthermore, some log entries pre-2010 have flags for the gblock type. For example:

+--------+----------+------------+----------------+---------------+---------------------------------+----------------+---------------------------+-------------+-----------+----------+
| log_id | log_type | log_action | log_timestamp  | log_namespace | log_title                       | log_comment_id | log_params                | log_deleted | log_actor | log_page |
+--------+----------+------------+----------------+---------------+---------------------------------+----------------+---------------------------+-------------+-----------+----------+
| 183748 | gblblock | gblock     | 20080821202044 |            -1 | Contributions/151.9.110.20      |        3593591 | 10 minutes
anon-only      |           0 |       696 |     NULL |
+--------+----------+------------+----------------+---------------+---------------------------------+----------------+---------------------------+-------------+-----------+----------+

The code currently does not handle this correctly.

Dreamy_Jazz renamed this task from Target of old GlobalBlocking log entries is the contributions page of the IP target, causing a InvalidArgumentException when CentralAuth is installed to Target of old GlobalBlocking log entries is the contributions page of the IP target and have flags defined, causing InvalidArgumentException and missing log parameter data.Tue, Apr 16, 5:43 PM
Dreamy_Jazz renamed this task from Target of old GlobalBlocking log entries is the contributions page of the IP target and have flags defined, causing InvalidArgumentException and missing log parameter data to GlobalBlockLogFormatter does not handle pre-2010 GlobalBlocking logs correctly, causing an InvalidArgumentException and missing log parameter data.Tue, Apr 16, 6:12 PM
Dreamy_Jazz moved this task from In Progress to Needs Review on the Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)) board.
Tchanders subscribed.Wed, Apr 17, 3:51 PM

Testing notes

I tested this by modifying an existing row with log_action gblock to contain similar data to the rows above:

UPDATE logging SET log_title = 'Contributions/1.2.3.4' WHERE log_id = 3854;
UPDATE logging SET log_params = '10 minutes\nanonymous only' WHERE log_id = 3854;

Then confirmed that there was no error, and that the log line substituted the correct message for the anon-only param.

gerritbot added a comment.Wed, Apr 17, 4:06 PM

Change #1020297 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@master] Handle pre-2010 gblblock log entries

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

ReleaseTaggerBot added a project: MW-1.43-notes (1.43.0-wmf.2; 2024-04-23).Wed, Apr 17, 5:00 PM
Dreamy_Jazz moved this task from Needs Review to Needs QA on the Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)) board.Wed, Apr 17, 5:16 PM
Dreamy_Jazz added a comment.EditedWed, Apr 17, 5:18 PM

I would suggest following @Tchanders testing steps for QA but also set the log_namespace to -1 using UPDATE logging SET log_namespace = -1 WHERE log_id = 3854;

This means that this ticket can only be QA'd on a local wiki. I've checked the betawikis and it seems that no log was made for the GlobalBlocking extension before 2012, so betawikis cannot be used to test this.

dom_walden subscribed.Thu, Apr 18, 3:19 PM

@Dreamy_Jazz a log_title of Contributions/71.107.128.0/18 leads to a log entry of ...globally blocked 18... where "18" is a link to "Special:Contributions/18".

thcipriani moved this task from Untriaged to Apr 2024 on the Wikimedia-production-error board.Thu, Apr 18, 3:25 PM
Dreamy_Jazz added a comment.EditedThu, Apr 18, 3:29 PM
In T362700#9726882, @dom_walden wrote:

@Dreamy_Jazz a log_title of Contributions/71.107.128.0/18 leads to a log entry of ...globally blocked 18... where "18" is a link to "Special:Contributions/18".

Thanks for noticing this.

That is not ideal. Better than exception, but still not fixed. I will move this out of QA while I work on a fix.

Dreamy_Jazz added a comment.Thu, Apr 18, 3:42 PM

On metawiki:

wikiadmin2023@10.64.136.13(metawiki)> select count(*) from logging where log_type = 'gblblock' and log_namespace = -1 and log_title LIKE 'Contributions/%';
+----------+
| count(*) |
+----------+
|      755 |
+----------+
1 row in set (0.010 sec)

wikiadmin2023@10.64.136.13(metawiki)> select count(*) from logging where log_type = 'gblblock' and log_namespace = -1;
+----------+
| count(*) |
+----------+
|      755 |
+----------+
1 row in set (15.514 sec)

This suggests that the fix for this can be to simply remove Contributions/ from the start of the title to get the IP.

gerritbot added a comment.Thu, Apr 18, 3:50 PM

Change #1021511 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@master] Handle pre-2010 logs for IP ranges

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

Dreamy_Jazz moved this task from Needs QA to Needs Review on the Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)) board.Thu, Apr 18, 3:50 PM
gerritbot added a comment.Thu, Apr 18, 3:53 PM

Change #1020936 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@REL1_42] Handle pre-2010 gblblock log entries

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

gerritbot added a comment.Thu, Apr 18, 4:02 PM

Change #1020936 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@REL1_42] Handle pre-2010 gblblock log entries

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

gerritbot added a comment.Thu, Apr 18, 4:23 PM

Change #1021511 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@master] Handle pre-2010 logs for IP ranges

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

gerritbot added a comment.Thu, Apr 18, 4:31 PM

Change #1020937 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@REL1_42] Handle pre-2010 logs for IP ranges

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

Dreamy_Jazz moved this task from Needs Review to Needs QA on the Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)) board.Thu, Apr 18, 4:32 PM
Dreamy_Jazz added a comment.Thu, Apr 18, 4:39 PM

Thanks again @dom_walden for finding that issue. The fix for that has now been merged, so QA should be able to continue again.

gerritbot added a comment.Thu, Apr 18, 4:40 PM

Change #1020937 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@REL1_42] Handle pre-2010 logs for IP ranges

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

dom_walden edited projects, added Trust and Safety Product Sprint (Sprint Pennywhistle (23rd April - 3rd May)); removed Trust and Safety Product Sprint (Sprint Tabla (1st - 14th April)).Tue, Apr 23, 10:50 AM
dom_walden moved this task from Priority Backlog to Needs QA on the Trust and Safety Product Sprint (Sprint Pennywhistle (23rd April - 3rd May)) board.
dom_walden moved this task from Needs QA to Done on the Trust and Safety Product Sprint (Sprint Pennywhistle (23rd April - 3rd May)) board.Tue, Apr 23, 1:20 PM

I have taken a sample of global blocking logs from the production metawiki database from different years and put them in my local database. I don't see any exceptions in the logs. The log lines in Special:Log look OK to me and the links work.

Test environment: local docker GlobalBlocking – (95ca8f1) 07:28, 22 April 2024.

Dreamy_Jazz closed this task as Resolved.Tue, Apr 23, 5:00 PM

Thanks for the thorough QA testing.

Maintenance_bot removed a project: Patch-For-Review.Fri, Apr 26, 5:54 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