Skip to content

fix(redis): use Upstash REST client#3705

Draft
RSO wants to merge 1 commit into
mainfrom
redis-upstash-rest-client
Draft

fix(redis): use Upstash REST client#3705
RSO wants to merge 1 commit into
mainfrom
redis-upstash-rest-client

Conversation

@RSO
Copy link
Copy Markdown
Contributor

@RSO RSO commented Jun 3, 2026

Summary

  • Replace the shared Redis helper's long-lived TCP client with the @upstash/redis REST client to avoid serverless socket readiness and command-queue stalls.
  • Support UPSTASH_REDIS_REST_URL/UPSTASH_REDIS_REST_TOKEN and Vercel KV KV_REST_API_URL/KV_REST_API_TOKEN env var pairs.
  • Update Redis-backed experiment cache tests to key off REST credentials instead of REDIS_URL.

Verification

N/A; backend cache helper change was not manually exercised against live Redis REST credentials.

Visual Changes

N/A

Reviewer Notes

Operators must configure either UPSTASH_REDIS_REST_URL + UPSTASH_REDIS_REST_TOKEN or KV_REST_API_URL + KV_REST_API_TOKEN before deploying this change. The existing REDIS_URL path remains in use by chat state and was not removed.

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.

1 participant