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

AO3-6440 Configure admin and user reset token expiry separately #4447

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

sarken
Copy link
Collaborator

@sarken sarken commented Feb 1, 2023

Issue

https://otwarchive.atlassian.net/browse/AO3-6440

Purpose

  • Add a separate ArchiveConfig.DAYS_UNTIL_ADMIN_RESET_PASSWORD_LINK_EXPIRES variable for configuring how long admin password reset tokens are valid.
  • Update both admin creation and admin password reset emails to mention the new value.
  • Also update the admin creation email to say what to do if the set password link doesn't work.

Testing Instructions

  • Create a new admin account for an email you have access to; ensure the email says the link is good for 5 days and the text is otherwise clear. Then delete said account.
  • Request a password reset and ensure the email says the link is good for 5 days.
  • Ideally wait 5 days to confirm the reset expires then and not in 7 days, but I think we're going to deploy before then, so we can double check on production.

Credit

Sarken, she/her

@sarken sarken added Priority: High - Broken on Test Merge immediately after approval Awaiting Review labels Feb 1, 2023
@redsummernight redsummernight changed the title AO3-6440 Configure admin and user reset token expiry seprately AO3-6440 Configure admin and user reset token expiry separately Feb 1, 2023
@redsummernight redsummernight merged commit 2d92ec1 into otwcode:master Feb 1, 2023
@sarken sarken deleted the AO3-6440 branch February 1, 2023 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants