upsy icon indicating copy to clipboard operation
upsy copied to clipboard

Use `ioredis`

Open kylemclaren opened this issue 1 year ago • 2 comments

This PR introduces a breaking change and replaces @upstash/redis with ioredis so that we can deploy the full stack on Fly. Upstash for Redis® on Fly.io uses a unified redis connection string for direct DB connections — not the REST API.

Now, instead of setting UPSTASH_REDIS_REST_URL and UPSTASH_REDIS_REST_TOKEN, we set a single REDIS_URL env var which looks like redis://[email protected]

kylemclaren avatar May 08 '24 17:05 kylemclaren

hey @kylemclaren have you checked the redis methods? I see you have not changed any method, so the new client is compatible with old one?

enesakar avatar May 08 '24 23:05 enesakar

@enesakar sorry I was OoO — I did briefly check the methods and they looked directly compatible. I left this in draft as I didn't have a chance to test it. Will try get to it this week!

kylemclaren avatar Jun 04 '24 08:06 kylemclaren