Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Email notify users when they are marked suspicious #557

Closed
chadwhitacre opened this issue Jan 29, 2013 · 9 comments
Closed

Email notify users when they are marked suspicious #557

chadwhitacre opened this issue Jan 29, 2013 · 9 comments

Comments

@chadwhitacre
Copy link
Contributor

chadwhitacre commented Jan 29, 2013

Send an email notification when possible to users when we mark them suspicious.


Original

This might be controversial.

I want to have a page at /about/fraud/untrusted/ which simply lists all participants with is_suspicious is true. As our fraud system evolves (#556), so will this listing. The principle is that untrusted users are announced as such so that false positives have a chance to discover this and attempt to redeem themselves. We can't be like PayPal, an infuriatingly opaque black box if you're a false positive.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@sigmavirus24
Copy link
Contributor

Perhaps, we also notify the user (if possible) and put a note on their
profile?

@mattbk
Copy link
Contributor

mattbk commented Oct 27, 2016

Perhaps, we also notify the user (if possible)

+1 to this. Recently interacted with a user at FD who couldn't log in and wasn't notified why--nothing happened when the user tried to log in. Took me a while to think of checking is_suspicious. Showing a message would help.

@mattbk
Copy link
Contributor

mattbk commented Nov 12, 2016

Perhaps, we also notify the user (if possible) and put a note on their
profile?

If you view the profile when not logged in, it has a pink overlay at the top and the word "Suspicious" below the avatar. That doesn't tell people what to do, however. And since they can't log in, showing them a notification doesn't help. I vote for sending an email when we mark someone suspicious.

mattbk added a commit that referenced this issue Nov 12, 2016
Part of #557. This covers the logic; it’s mostly a duplicate of
`dashboard/index` with the buttons removed (admins can mark/unmark
suspicious on the profile itself).

Obviously needs to be put into a site template for ~users to see, and
only needs the list, not the other frame.
@chadwhitacre
Copy link
Contributor Author

Recently interacted with a user at FD who couldn't log in and wasn't notified why--nothing happened when the user tried to log in.

For the record, the user turned out to be a true positive. ;-)

@chadwhitacre
Copy link
Contributor Author

Perhaps, we also notify the user (if possible) and put a note on their profile?

As discussed at #4188 (comment), I like this solution better than the listing of suspicious accounts, because a) it's much less work for those affected to discover that they're affected, and b) it's more respectful of user privacy. A public listing would only seem to make sense if we're trying to warn others about these users or something like that, but that doesn't seem necessary since suspicious users can't do anything on the site (so there's nothing to watch out for).

@mattbk
Copy link
Contributor

mattbk commented Nov 21, 2016

I'm fine with that, but I think there should be a message (not just a note on the profile) when someone tries to log in and are denied because they're suspicious. My first inkling in such a case would be that the login form is broken, not that I should go look at my own profile page to see what's up.

@chadwhitacre chadwhitacre changed the title add a page that shows untrusted users Email notify users when they are marked suspicious Nov 21, 2016
@chadwhitacre
Copy link
Contributor Author

I'm fine with that, but I think there should be a message (not just a note on the profile) when someone tries to log in and are denied because they're suspicious.

Okay. I've rescoped this to be about email notification, and reticketed the login prompt as #4199.

@mattbk
Copy link
Contributor

mattbk commented Apr 7, 2017

@chadwhitacre
Copy link
Contributor Author

Search for app.email_queue.put.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants