Skip to content

build: propagate ring/aws-lc-rs feature flags to bollard#781

Merged
DDtKey merged 3 commits intotestcontainers:mainfrom
kaitlinsm:main
Mar 24, 2025
Merged

build: propagate ring/aws-lc-rs feature flags to bollard#781
DDtKey merged 3 commits intotestcontainers:mainfrom
kaitlinsm:main

Conversation

@kaitlinsm
Copy link
Copy Markdown
Contributor

In order for consumers to be able to switch between ring and aws-lc-rs cleanly without testcontainers_rs bringing in the wrong library, add feature flags that are propagated through to bollard to switch between the two ssl providers.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 24, 2025

Deploy Preview for testcontainers-rust ready!

Name Link
🔨 Latest commit 0c09975
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-rust/deploys/67e1776d35899d000877803c
😎 Deploy Preview https://deploy-preview-781--testcontainers-rust.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@DDtKey
Copy link
Copy Markdown
Contributor

DDtKey commented Mar 24, 2025

Also test suite relies on cargo hack. We need to adjust CI to have ssl feature enabled

@DDtKey
Copy link
Copy Markdown
Contributor

DDtKey commented Mar 24, 2025

Thank you for the contribution 👍

@DDtKey
Copy link
Copy Markdown
Contributor

DDtKey commented Mar 24, 2025

LGTM 🚀

Later we will need to think about optimization of CI, because matrix is increased significantly (9 -> 31 use-case, ~6m -> 17m). Not a blocker for this though

Thank you one more time!

@DDtKey DDtKey merged commit 8168a82 into testcontainers:main Mar 24, 2025
12 checks 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.

3 participants