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

Support Redis 7 #4988

Open
rdcwaldrop1 opened this issue May 2, 2023 · 4 comments
Open

Support Redis 7 #4988

rdcwaldrop1 opened this issue May 2, 2023 · 4 comments

Comments

@rdcwaldrop1
Copy link

Is your feature request related to a problem? Please describe.

We would like to upgrade our Elasticache Redis to version 7, but currently the Tyk Gateway is using github.com/go-redis/redis/v8 v8.11.5 which is incompatible with this Redis version. This is preventing us from taking advantage of the latest security and performance enhancements in Redis.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Bump the version of go-redis to v9 as suggested in this GitHub issue redis/go-redis#2085

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Until this is implemented we will keep our Redis version in 6.x

Additional context
Add any other context or screenshots about the feature request here.

The error we receive when trying to communicate with a Redis 7 cluster is as follows:

time="May 02 20:14:05" level=error msg="Could not send notification: got 4 elements in cluster info address, expected 2 or 3" prefix=pub-sub
@rdcwaldrop1
Copy link
Author

I'm happy to create a PR so this change can be tested. Let me know if you'd like me to add this.

@sedkis
Copy link
Contributor

sedkis commented May 3, 2023

Fixed by #4211

@matthewdickinson
Copy link

Just checking in on this, is there a timeline for Redis 7 support? I see there have been a few PRs, but all the ones I found have been closed and not merged.

Thanks!

@caroltyk
Copy link

Hi @matthewdickinson , yes we are working on Redis 7 support and it will be coming soon in Q1.

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

No branches or pull requests

5 participants