Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FP submitted through API doesn't remove user from blacklist #346

Closed
makyen opened this issue Mar 10, 2018 · 8 comments · Fixed by #473
Closed

FP submitted through API doesn't remove user from blacklist #346

makyen opened this issue Mar 10, 2018 · 8 comments · Fixed by #473
Labels
type: bug Buggy or otherwise incorrect behaviour.

Comments

@makyen
Copy link
Contributor

makyen commented Mar 10, 2018

Currently, if you submit a fp in chat, the user is automatically removed from the blacklist. However, if you submit a fp through the API (e.g. FIRE), then the user is not removed from the blacklist.

@angussidney
Copy link
Member

See also: Charcoal-SE/SmokeDetector#1072

Although it's not worded in the best way, they're both asking for FP API feedbacks to remove the user from the blacklist.

@angussidney angussidney added the type: bug Buggy or otherwise incorrect behaviour. label Mar 10, 2018
@thesecretmaster
Copy link
Member

So, is this something that should be fixed in MS? It seems like a SmokeDetector issue to me.

@makyen
Copy link
Contributor Author

makyen commented Mar 15, 2018

@thesecretmaster I'm fine with wherever it's appropriate.

Given that my issue is that I want the metasmoke API to be changed, I had assumed it went in the metasmoke repository. I don't really know how the code is partitioned, but I'd assumed it was the "metasmoke API", not the "SmokeDetector API", in part, to indicate that it was in this project/repository.

@quartata
Copy link
Member

quartata commented Mar 15, 2018

This couldn't be consistently done with Metasmoke changes (if the post had been recently reported, we'd have the owner URL on hand and could use that but we aren't guaranteed)

@ArtOfCode-
Copy link
Member

Completed

@makyen
Copy link
Contributor Author

makyen commented Apr 5, 2018

This does not appear to have been fixed. Or at least I believe MS has deployed more recently than 10 days ago. See this instance of SD indicating that a user still needed to be manually removed from the blacklist when they had been given 3 fp feedbacks from two people on two different reports, both of which the only reason was that the user was blacklisted.

@j-f1
Copy link
Contributor

j-f1 commented Apr 8, 2018

The code status page says metasmoke is running the latest commit.

@angussidney angussidney reopened this Apr 11, 2018
@tripleee
Copy link
Member

Tangentially, it would be nice if there was a way to say "this was an FP but I don't want this troll to be removed from the blacklist just yet".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Buggy or otherwise incorrect behaviour.
Development

Successfully merging a pull request may close this issue.

7 participants