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

VerificationToken - Expiration time added in argument #42356

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

theronakpatel
Copy link

@theronakpatel theronakpatel commented Dec 18, 2023

Summary

We have a requirement to set token expiration at specific intervals, like 15 minutes or 1 day. To address this, I am suggesting an enhancement.

TODO

  • ...

Checklist

@skjnldsv skjnldsv added the 2. developing Work in progress label Feb 21, 2024
@susnux susnux added this to the Nextcloud 30 milestone Apr 18, 2024
This was referenced Jul 30, 2024
@joshtrichards
Copy link
Member

I presume the original idea was to make the duration configurable, correct? This PR seems to just moves it to a hardcoded variable not a configuration option. Am I overlooking something?

This was referenced Aug 5, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 13, 2024
@skjnldsv skjnldsv removed this from the Nextcloud 30 milestone Aug 14, 2024
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.

4 participants