Page MenuHomePhabricator

Using Nuke on an IP address on a wiki without temporary accounts results in an edit summary referencing temporary accounts
Closed, ResolvedPublic3 Estimated Story Points

Description

User Svartava used Nuke on an IP address (see example log at https://meta.wikimedia.org/w/index.php?title=Special:Log&logid=59338121) on Meta, where temporary accounts are not currently deployed. This seems to have resulted in an edit summary referencing temporary accounts, which should not be the case.

If a wiki isn't using temporary accounts, the same edit summary used for accounts should be used, as I believe was the case prior to adding temporary accounts support to Nuke.

Event Timeline

I went to test this on en.wiki and couldn't reproduce - entering my IPv6 after creating a test page resulted in the standard message.

image.png (1×2 px, 150 KB)

I can reproduce the error on meta. Perhaps it's because temporary accounts are "known" on meta, even though they are not enabled (T376132), while they are neither enabled nor known on enwiki?

Screenshot 2025-04-02 at 10.16.29.png (756×1 px, 105 KB)

DMburugu triaged this task as Medium priority.Apr 15 2025, 4:28 PM
Scardenasmolinar changed the task status from Open to In Progress.Apr 16 2025, 3:06 AM
Scardenasmolinar claimed this task.
Scardenasmolinar moved this task from Ready to In Progress on the Moderator-Tools-Team (Kanban) board.

Change #1139579 had a related patch set uploaded (by Scardenasmolinar; author: Scardenasmolinar):

[mediawiki/extensions/Nuke@master] Fix edit summary referencing temp accounts

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

Change #1139579 merged by jenkins-bot:

[mediawiki/extensions/Nuke@master] Fix edit summary referencing temp accounts

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

It's worth pointing out that in the ticket that was merged I also reported the separate bug that you have to accept the temporary account agreement to nuke legacy IP edits. It looks like that patch is at a high enough level of abstraction it should fix that too, though.

@Pppery and @Johannnes89, can you confirm that this no longer happens on Meta when Nuking pages?

for reference, meta is the only wiki with

$wgAutoCreateTempUser['enabled'] = true;
$wgAutoCreateTempUser['known'] = false;

which is controlled by $wmgDisableIPMasking

The edit summary is correct now.
I didn't check the related bug about having to accept the temporary account agreement to nuke legacy IP edits, because my account (as a steward) is not required to accept it.

I can confirm that the temp account agreement bug is fixed too.

jsn.sherman moved this task from QA to Done on the Moderator-Tools-Team (Kanban) board.

I can confirm that the temp account agreement bug is fixed too.

Thanks for the QA!