Skip to content

Do not ignore federated reports targetting already-reported accounts#9534

Merged
Gargron merged 1 commit intomastodon:masterfrom
ClearlyClaire:fixes/federated-reports
Dec 30, 2018
Merged

Do not ignore federated reports targetting already-reported accounts#9534
Gargron merged 1 commit intomastodon:masterfrom
ClearlyClaire:fixes/federated-reports

Conversation

@ClearlyClaire
Copy link
Copy Markdown
Contributor

No description provided.

@ClearlyClaire
Copy link
Copy Markdown
Contributor Author

So, currently, only the first remote report from a given account targeting a local account is processed.

This means a high-quality report sent after a low-quality one will be lost. It also means that a repeat offender will only get reports from their first offense (indeed, new reports will be ignored if there is an existing report, resolved or not).

This is made worse by the fact that all reports from a same instance will typically be sent from the same account.

Copy link
Copy Markdown
Member

@nightpool nightpool left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thibg's reasoning is sound here, I would like to see this merged. I didn't even realize that the reports I was seeing might be incomplete or not reflect the actual quantity of reports. For example, if someone gets reported for not marking their media as NSFW, and that issue is resolved, then I would never see a report when 6 months later they decide to go on a racist tirade and i need to ban them. This is super exploitable by bad actors.

@Gargron Gargron merged commit b2f4114 into mastodon:master Dec 30, 2018
@ClearlyClaire ClearlyClaire deleted the fixes/federated-reports branch March 14, 2019 15:41
hiyuki2578 pushed a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants