Skip to content

feat(libstore): add curl-based S3 store implementation#14198

Merged
xokdvium merged 1 commit intoNixOS:masterfrom
lovesegfault:curl-based-s3-pieces
Oct 10, 2025
Merged

feat(libstore): add curl-based S3 store implementation#14198
xokdvium merged 1 commit intoNixOS:masterfrom
lovesegfault:curl-based-s3-pieces

Conversation

@lovesegfault
Copy link
Member

Motivation

Add a new S3BinaryCacheStore implementation that inherits from
HttpBinaryCacheStore.

The implementation is activated with NIX_WITH_CURL_S3, keeping the
existing NIX_WITH_S3_SUPPORT (AWS SDK) implementation unchanged.

Context

Carve out of #13752


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@github-actions github-actions bot added the store Issues and pull requests concerning the Nix store label Oct 9, 2025
@lovesegfault lovesegfault force-pushed the curl-based-s3-pieces branch 4 times, most recently from 1209c1b to e0910b1 Compare October 9, 2025 22:08
@Ericson2314 Ericson2314 requested a review from xokdvium October 9, 2025 22:14
Copy link
Member

@Ericson2314 Ericson2314 left a comment

Choose a reason for hiding this comment

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

It looks good from me, but I would appreciate @xokdvium as another set of eyes

@lovesegfault lovesegfault force-pushed the curl-based-s3-pieces branch 3 times, most recently from c8d3c41 to 7701bb3 Compare October 10, 2025 00:08
Add a new S3BinaryCacheStore implementation that inherits from
HttpBinaryCacheStore.

The implementation is activated with NIX_WITH_CURL_S3, keeping the
existing NIX_WITH_S3_SUPPORT (AWS SDK) implementation unchanged.
@xokdvium xokdvium added this pull request to the merge queue Oct 10, 2025
Merged via the queue into NixOS:master with commit 2308aaf Oct 10, 2025
18 checks passed
@edolstra edolstra mentioned this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants