Skip to content

Add SSL/TLS support for Redis connections - #1580

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

Add SSL/TLS support for Redis connections#1580
BenCodez merged 1 commit into
masterfrom
feature/redis-tls

Conversation

@BenCodez

@BenCodez BenCodez commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • add opt-in Redis.SSL support for backend (Paper/Spigot), BungeeCord, and Velocity Redis connections
  • apply TLS consistently to both Redis subscribers and the proxy's dedicated publisher pool
  • add MultiProxyRedis.SSL for separate multi-proxy Redis connections
  • preserve existing plaintext behavior by default
  • validate Redis certificates with the JVM truststore and verify hostnames
  • document the new settings in the bundled YAML configurations
  • add unit coverage for TLS-enabled and default plaintext client configuration

Configuration

Redis:
  SSL: true

For a separate multi-proxy connection:

MultiProxyRedis:
  SSL: true

When MultiProxyRedis.UseExistingConnection is true, the primary Redis.SSL value applies.

Verification

  • mvn test on Java 21: 297 tests passed, 0 failures, 0 errors
  • both YAML resource files parse successfully
  • whitespace validation passed
  • dynamic CodeQL analysis completed successfully

Dependency

Depends on BenCodez/SimpleAPI#72. Merge and publish/rebuild that snapshot before building this 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 3354cc7 into master Aug 26, 2026
4 of 5 checks passed
@BenCodez
BenCodez deleted the feature/redis-tls branch August 26, 2026 23:22
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