Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add redis tls feature 'rediss://' #130

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

mguegan
Copy link
Contributor

@mguegan mguegan commented May 16, 2024

Update to redis crate 0.25.2 with TLS support

This should fix #121

Tested OK on Azure Cache for Redis

@@ -15,13 +15,13 @@ FROM rust:1.76-alpine as backend
WORKDIR /tmp
RUN apk add --no-cache libc-dev openssl-dev alpine-sdk
COPY ./packages/backend ./
RUN cargo build --release
RUN RUSTFLAGS="-Ctarget-feature=-crt-static" cargo build --release
Copy link
Owner

Choose a reason for hiding this comment

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

What is this for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The rust alpine image using libmusl can't build the binary without explicitly telling him to use shared libraries (by setting crt-static equals false). But I'm not a Rust expert, maybe you can find another way around.

@cupcakearmy cupcakearmy merged commit 0d9f3fe into cupcakearmy:main Aug 22, 2024
1 check passed
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.

'rediss//' does not work (double 'ss' for TLS)
2 participants