Last modified: 2014-04-07 22:54:04 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 T60016, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58016 - Flow: Suppression redacts the wrong username
Flow: Suppression redacts the wrong username
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
unspecified
All All
: Highest normal (vote)
: ---
Assigned To: Nobody - You can work on this!
flow_suppression
:
Depends on:
Blocks: 60178
  Show dependency treegraph
 
Reported: 2013-12-05 01:58 UTC by Quiddity
Modified: 2014-04-07 22:54 UTC (History)
11 users (show)

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


Attachments
Suppression still redacting wrong username on mediawiki.org (450.59 KB, image/png)
2014-03-24 19:07 UTC, Dan Garry
Details

Description Quiddity 2013-12-05 01:58:53 UTC
/Username suppressed/ is Quiddity (the suppressor).
The IP is still visible. 
>.>
Comment 1 Bingle 2013-12-05 02:31:36 UTC
The WMF core features team tracks this bug on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/587, but people from the community are welcome to contribute here and in Gerrit.
Comment 2 Matthias Mullie 2013-12-17 17:00:18 UTC
This is in history, I think?

While we do indeed redact the wrong username, shouldn't we just remove any trace of entries that have been suppressed, for users that don't have permissions to see them? (just like suppression log isn't visible to all users)

And for users that do have sufficient permissions to view suppressed entries, we can probably show the them with their normal, non-redacted, usernames?
Comment 3 Gerrit Notification Bot 2013-12-17 18:46:43 UTC
Change 102203 had a related patch set uploaded by Matthias Mullie:
Fetch moderation status from last revision

https://gerrit.wikimedia.org/r/102203
Comment 4 Quiddity 2013-12-17 19:27:25 UTC
(In reply to comment #2)
> This is in history, I think?

I saw it in Recent Changes, but everywhere that it might appear needs to be checked.
Comment 5 Gerrit Notification Bot 2013-12-31 19:22:49 UTC
Change 102203 merged by jenkins-bot:
Fetch moderation status from last revision

https://gerrit.wikimedia.org/r/102203
Comment 6 Erik Bernhardson 2013-12-31 19:39:23 UTC
The above change is a partial fix, but not really fixing the base issue.
Comment 7 Greg Grossmeier 2014-01-27 04:58:20 UTC
Erik B: What more is needed? What is still wrong?
Comment 8 Matthias Mullie 2014-01-27 18:18:11 UTC
Greg: There may be more, but as far as I know:

We currently look at the most recent revision to see if the user has permissions to see that. If it's suppressed & user has no privileges, we don't show anything about that post at all.

However, if a post is suppressed, then restored (so most recent revision != suppressed), an unprivileged user will see a history entry about suppression (probably still with the incorrect redacted name), but he should probably not be allowed to see the suppress & restore history lines.
Comment 9 Risker 2014-02-04 05:53:37 UTC
Okay, on testing suppression on https://en.wikipedia.org/wiki/Wikipedia_talk:Flow/Developer_test_page 

The following post was suppressed as a test:

 Risker (talk | contribs | block)	

Titodutta: thank you for your post. I am testing again because I have reported "quivering" text. With each keystroke, everything from this editing box on down quivers. But everything above it remains stable.

*as an oversighter, the post does not disappear for me at all, except for the username of the contributor (which in this case is me). I did not think my suppression was successful, since the content of the post is visible in exactly the way it existed before the suppression. It took several reviews for me to identify even that the username was not visible.
*I have had a non-oversighter look at the page and they cannot see the post or the username. They cannot see that a post was removed.

*The suppression also shows up in my watchlist.
Comment 10 Kunal Mehta (Legoktm) 2014-02-04 06:27:40 UTC
(In reply to comment #9)

> *The suppression also shows up in my watchlist.

Filed as bug 60814.
Comment 11 Maryana Pinchuk 2014-02-24 17:52:31 UTC
Matthias has submitted a number of patches to resolve the issues listed here, and we had Risker, Deskana, and AGK (all oversighters from enwiki) test things out on ee-flow last week.

To summarize the current behavior on ee-flow:

* As a normal user (no advanced rights), in RC and history, I can see a crossed-out entry indicating that something was suppressed but *not* the content of the suppressed entry, the title of the suppressed entry, or the username of the user who submitted the suppressed entry (or the person who suppressed it – this, as far as I can understand, is a separate bug).

* As a user with oversighter rights, I can:
** Suppress topics and posts from the Flow board. When I do this, I get a confirmation message telling me the action was successful, and the content of the topic/post is hidden under a marker indicating that the topic/post was suppressed, which lets me know that the content is suppressed.
** See the diff of suppressed entries by following the diff link from RC/history.
** See the user who submitted the suppressed content and the user who suppressed it in the suppression log.

Please double-check to make sure I've summarized this appropriately - if so, I'll close this as fixed and we'll send it out to all wikis during our next LD window :)

Username suppression is a separate bug I need to log, but it looks like the behavior we have now, while not perfect, is not revealing any sensitive information to users without the appropriate rights.
Comment 12 Dan Garry 2014-02-25 01:34:28 UTC
(In reply to Maryana Pinchuk from comment #11)
> * As a normal user (no advanced rights), in RC and history, I can see a
> crossed-out entry indicating that something was suppressed but *not* the
> content of the suppressed entry, the title of the suppressed entry, or the
> username of the user who submitted the suppressed entry (or the person who
> suppressed it – this, as far as I can understand, is a separate bug).

The suppression log entry (i.e. the bit that contains the name of the oversighter) should not appear in the recent changes feed, history or watchlist of any page. It should *only* appear in the suppression log. The same is true of any unsuppression log entries.

This means that the actual bug is that a normal user can see the log entry at all, as it should just be in the private suppression log. The fact the username of the suppressor is itself suppressed is a separate bug from that.
 
> * As a user with oversighter rights, I can:
> ** Suppress topics and posts from the Flow board. When I do this, I get a
> confirmation message telling me the action was successful, and the content
> of the topic/post is hidden under a marker indicating that the topic/post
> was suppressed, which lets me know that the content is suppressed.
> ** See the diff of suppressed entries by following the diff link from
> RC/history.
> ** See the user who submitted the suppressed content and the user who
> suppressed it in the suppression log.

That's all correct.

> Please double-check to make sure I've summarized this appropriately - if so,
> I'll close this as fixed and we'll send it out to all wikis during our next
> LD window :)

8 hours is not a very long period of time to give people to respond! :-)
Comment 13 Dan Garry 2014-03-24 19:07:02 UTC
Created attachment 14890 [details]
Suppression still redacting wrong username on mediawiki.org
Comment 14 Dan Garry 2014-03-24 19:09:31 UTC
Reopening since this bug still exists on mediawiki.org. See attachment 14890 [details] for verification.

Noting as before that the actual bug here is that the log entry is visible at all; only users with oversight rights should see any log entries relating to suppression.
Comment 15 Matthias Mullie 2014-03-27 12:46:21 UTC
Well, I don't know here... Seeing that log entry is to be expected according to how we implemented things.

It looks like the thing that got suppressed was the topic title. You're seeing a log entry for a comment (that was not suppressed - I guess) on that topic, with the topic name replaced by a message that it's suppressed.

That message should indeed be fixed to display the real username (actually, I think a fix of that has recently been submitted - I'll check that out).

As for if the log entry should actually be displayed: I don't know. Right now, suppressing a topic title only suppressed exactly that; all posts in the topic are left untouched (same for suppressing a post that has replies - those replies are left alone)

Should suppressing a topic title suppress everything within that topic (and suppressing a post suppress all child replies), or not?
Or should we build something new to make both possible?
Comment 16 Gerrit Notification Bot 2014-03-27 15:23:18 UTC
Change 121385 had a related patch set uploaded by Matthias Mullie:
Don't escape the userlinks displayed in "This topic was suppressed by <links>"

https://gerrit.wikimedia.org/r/121385
Comment 17 Gerrit Notification Bot 2014-04-04 19:32:58 UTC
Change 121385 merged by jenkins-bot:
Don't escape the userlinks displayed in "This topic was suppressed by <links>"

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

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


Navigation
Links