Skip to content

feat(services/redis): enable TLS#2670

Merged
Xuanwo merged 1 commit intoapache:mainfrom
Stormshield-robinc:redis_tls
Jul 20, 2023
Merged

feat(services/redis): enable TLS#2670
Xuanwo merged 1 commit intoapache:mainfrom
Stormshield-robinc:redis_tls

Conversation

@Stormshield-robinc
Copy link
Copy Markdown
Contributor

enable TLS support for redis since redis-rs now support rustls
#2669

Note : in the sccache documentation for redis (https://github.com/mozilla/sccache/blob/main/docs/Redis.md) it seem that it was possible to add a #insecure to url. Is this something you would be interested in ?

@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jul 19, 2023
Comment thread core/src/services/redis/backend.rs
Comment thread core/Cargo.toml Outdated
@Stormshield-robinc Stormshield-robinc marked this pull request as draft July 19, 2023 15:16
@Stormshield-robinc Stormshield-robinc marked this pull request as ready for review July 19, 2023 16:45
Comment thread core/Cargo.toml Outdated
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jul 20, 2023

Thanks!

@Xuanwo Xuanwo merged commit 79ffdae into apache:main Jul 20, 2023
@Stormshield-robinc
Copy link
Copy Markdown
Contributor Author

Thanks for your reviews.

@Stormshield-robinc Stormshield-robinc deleted the redis_tls branch July 20, 2023 17:12
@oowl oowl mentioned this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants