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

[BUG] Upstash memory key TTL dosen't change even though Session Timeouts is set in the Upstash node #2255

Closed
alvinho917 opened this issue Apr 25, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@alvinho917
Copy link

alvinho917 commented Apr 25, 2024

Describe the bug
I set the session timeouts in the Upstash node to be 120 and I also send "sessionTTL": 120 as overrideConfig while sending request. However, in Upstash console the TTL of the key is not changed and is still Forever

To Reproduce
Steps to reproduce the behavior:

  1. Create Upstash Redis-Backed Chat Memory node
  2. Click on 'Additional Parameters' and input value for the Session Timeouts
  3. Connect Upstash node to conversation chain
  4. Send API request and add "sessionTTL": 120 as overrideConfig
  5. Open Upstash console, find the key and the TTL is still forever
  6. Wait for 120 seconds and the key is still not deleted

Expected behavior
The TTL of the key should be the same as the Session Timeouts set in the Upstash node. The key automatically delete itself after the timeout.

Screenshots
image
image
image

Flow
Test Chatflow.json

Additional context
I am trying to set the TTL because I find out that if an user saved too many memory, it makes the request exceed the maximum token value of the AI model. So I wanna ask is setting TTL for the keys the best way to solve this problem?

@HenryHengZJ HenryHengZJ added the bug Something isn't working label Apr 29, 2024
@HenryHengZJ
Copy link
Contributor

Fixed: #2871

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants