Skip to content

feat(cache) TLS config - #948

Open
Strnadj wants to merge 2 commits into
PrivadoID:developfrom
Strnadj:feature/allow_tls_cache
Open

feat(cache) TLS config#948
Strnadj wants to merge 2 commits into
PrivadoID:developfrom
Strnadj:feature/allow_tls_cache

Conversation

@Strnadj

@Strnadj Strnadj commented Nov 27, 2025

Copy link
Copy Markdown

No description provided.

@Strnadj
Strnadj requested a review from a team as a code owner November 27, 2025 11:30
@x1m3

x1m3 commented Nov 27, 2025

Copy link
Copy Markdown
Contributor

Hi @Strnadj , and thank you very much for this PR!

We really appreciate you taking the time to contribute. According to our contribution guidelines (contrib.md), all pull requests should target the devel branch instead of main, so that we can keep main stable.

Could you please reopen this PR (or open a new one) against the devel branch?

@x1m3

x1m3 commented Nov 27, 2025

Copy link
Copy Markdown
Contributor

Regarding your PR, have you tried to pass redis url starting with rediss like "rediss://:password@host:6379/0" ? It should work and it is the simple way to configure a secure connection without adding more config params

@Strnadj
Strnadj changed the base branch from main to develop November 27, 2025 12:19
@Strnadj

Strnadj commented Nov 27, 2025

Copy link
Copy Markdown
Author

Regarding your PR, have you tried to pass redis url starting with rediss like "rediss://:password@host:6379/0" ? It should work and it is the simple way to configure a secure connection without adding more config params

For redis maybe, (doc says something else https://redis.uptrace.dev/guide/go-redis.html#using-tls) I can check that later, but valkey has no option like this...

@Strnadj

Strnadj commented Nov 28, 2025

Copy link
Copy Markdown
Author

@x1m3 I was able to test rediss and it's working... so I will remove that.

Related base branch it has been changed to devel.

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.

2 participants