-
Notifications
You must be signed in to change notification settings - Fork 540
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
Support for time limited links #345
Comments
@JanC89 we will look into it and add it to our backlog. Also, we open for PRs and would gladly review a PR. Thank you for your effort. |
This issue has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled. |
This issue was closed because it has been inactive for 7 days since being marked as closing-soon. |
This could be an important feature. |
Describe the feature you'd like
Add support for time limited links. The solution already supports signature validation. So it would be great if you could add an
Expires
parameter to the URL that will be part of the signing. The Lambda will then simply check the current timestamp against the timestamp specified. If the link has expired return a403 Forbidden
Our company is very interested in this feature, so we would gladly make an initial PR. We would however like some feedback before we do it, to ensure it is something that fits your pipeline/roadmap.
Thanks!
The text was updated successfully, but these errors were encountered: