Skip to content

fix(redis): add redis_server_name configuration - #13935

Closed
Arjen10 wants to merge 3 commits into
apache:masterfrom
Arjen10:fix/redis-tls-sni
Closed

Arjen10 wants to merge 3 commits into
apache:masterfrom
Arjen10:fix/redis-tls-sni

Conversation

@Arjen10

@Arjen10 Arjen10 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Description

Implemented two changes:

  1. Added the redis_server_name configuration for Redis connections.
  2. Added redis_server_name to the Redis connection pool.

Which issue(s) this PR fixes:

Fixes #13926

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@Arjen10 Arjen10 changed the title fix(redis): redis add redis_server_name conf fix(redis): add redis_server_name configuration Sep 14, 2026
@Arjen10

Arjen10 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

This PR only fixes TLS SNI for standalone Redis (policy: redis), which is the path described in #13926.

policy: redis-cluster uses a different client. Cluster SNI is out of scope here and should be handled in a follow-up PR.

@Arjen10

Arjen10 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

Closing this as #13938 has been merged for #13926.

The first commit on that PR came from this branch. I would have preferred the follow-ups to land here, but there is nothing left to review on this side.

@Arjen10 Arjen10 closed this Sep 15, 2026
@Arjen10
Arjen10 deleted the fix/redis-tls-sni branch September 15, 2026 08:12
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.

bug: Redis TLS connections should set SNI (server_name) to redis_host when redis_ssl is enabled

1 participant