Skip to content

Add SSL/TLS support to RedisHandler - #72

Merged
BenCodez merged 1 commit into
mainfrom
feature/redis-tls
Aug 26, 2026
Merged

Add SSL/TLS support to RedisHandler#72
BenCodez merged 1 commit into
mainfrom
feature/redis-tls

Conversation

@BenCodez

@BenCodez BenCodez commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • add an SSL/TLS-aware RedisHandler constructor while preserving the existing plaintext constructor
  • configure Jedis TLS through DefaultJedisClientConfig
  • enable certificate hostname verification with the HTTPS endpoint-identification algorithm
  • rely on the JVM truststore for Redis certificate validation
  • add tests for both TLS-enabled and default plaintext client configurations

Verification

  • mvn test on Java 21: 168 tests passed, 0 failures, 0 errors

Follow-up

VotingPlugin integration is in BenCodez/VotingPlugin#1580. Merge and publish this SimpleAPI snapshot before building that PR.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for security reviews. Please try again later.

@BenCodez
BenCodez merged commit 23a5665 into main Aug 26, 2026
4 checks passed
@BenCodez
BenCodez deleted the feature/redis-tls branch August 26, 2026 23:18
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