Skip to content

Conversation

@tommy-couzens
Copy link
Contributor

These options accept true and false, lowercase, but the docs currently say TRUE and FALSE.

When I set to TRUE, I see the USE TLS button not ticked:
image

When I set to true, it is ticked
image

@CLAassistant
Copy link

CLAassistant commented Oct 27, 2025

CLA assistant check
All committers have signed the CLA.

@jit-ci
Copy link

jit-ci bot commented Oct 27, 2025

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. How about you, @ViktarStarastsenka?

@dwdougherty
Copy link
Collaborator

Thank you, @tommy-couzens. Before I can merge this PR, you need to sign our CLA.

@tommy-couzens
Copy link
Contributor Author

@dwdougherty thanks for the quick response, I have signed

@dwdougherty
Copy link
Collaborator

Thanks. I'll just give @ViktarStarastsenka a chance to review before I merge.

@dwdougherty dwdougherty merged commit 63f06ed into redis:main Oct 29, 2025
5 checks passed
@ViktarStarastsenka
Copy link
Contributor

@tommy-couzens , thank you for noticing!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants