-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Add task in order to delete unconfirmed users (older than 2days) #1571
Conversation
lib/tasks/mastodon.rake
Outdated
namespace :users do | ||
desc 'clear unconfirmed users' | ||
task clear: :environment do | ||
User.where('confirmed_at is NULL AND confirmation_sent_at <= ?', 7.days.ago).find_each(&:destroy) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm maybe 24 hours or 2 days? Highly unlikely that someone would press "confirm" in an e-mail after waiting for so long.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, 2 days seems good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, add a line on the first mail or on the web page : "You have two days for validate your account..."
Can you update the documentation on this one? Some administration document or similar, perhaps? |
Hello, From what I can see in the diff it is just deleting the user from DB, is there any plan to change/improve that ? |
@Fastidious |
@symac You shouldn't be able to discover unconfirmed users, since they can't login (therefore, can't post) Suspended accounts return 410. |
Can you release v1.1.2 with this merge ? Thanks a lot. |
* add task in order to delete unconfirmed user * change 7 days to 2
Related to #1565
just add a new rake task in order to delete unconfirmed users