-
-
Notifications
You must be signed in to change notification settings - Fork 1
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 years, minutes, and seconds as expiration times #94
Conversation
e3b5fc3
to
d26fdf3
Compare
Hm, the test passes locally... I'll investigate. |
Uff, seems to be a regression in core105 - it still passed with core101. |
Ah, not a regression, but a fix :D the test had a workaround for deltachat/deltachat-core-rust#3650, which is not necessary anymore and causes the test to fail now. Removing it fixes it. I opened a separate PR: #97 |
d26fdf3
to
89a8435
Compare
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.
if you also add a "y" test it's fine to merge.
closes #77
The main purpose of this PR is handling some cases in which users might add unexpected input - seconds will not have much use in production, as prune() is only executed every 10 minutes. But it's still interesting to keep around, as developers can trigger prune() earlier by restarting mailadm or running
mailadm prune
manually, so it's useful for testing purposes even if we can't guarantee that users are indeed created 10 seconds after their creation if their token is only giving them 10 seconds.