Charcoal-SE / metasmoke

Web dashboard for SmokeDetector.
https://metasmoke.erwaysoftware.com
Creative Commons Zero v1.0 Universal
43 stars 34 forks source link

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

Closed makyen closed 6 years ago

makyen commented 6 years ago

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 commented 6 years ago

See also: https://github.com/Charcoal-SE/SmokeDetector/issues/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.

thesecretmaster commented 6 years ago

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

makyen commented 6 years ago

@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 commented 6 years ago

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- commented 6 years ago

Completed

makyen commented 6 years ago

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 commented 6 years ago

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

tripleee commented 6 years ago

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".