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

Optional userinfo ttl #671

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

richhaase
Copy link

Issue #, if available:

Description of changes:

This PR adds the ability to set a TTL on records added to the DynamoDB UsersTable by adding:

  • a TimeToLiveSpecification to the UserTable setting the TTL field to ttl
  • a setting called USERINFO_TTL_DAYS
  • functionality to the fulfillment lambda which causes it to set an appropriate ttl on user cache records every time a new interaction occurs, IF the USERINFO_TTL_DAYS setting is greater than zero

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

By default this will be set to 0, which leaves the feature disabled.
Sets the ttl on a user cache record to DYNAMODB_TTL_DAYS in the future.
@marcburnie
Copy link
Member

Thanks @richhaase for contributing! This looks like a useful feature. We will review and get back to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants