Skip to content

Conversation

@peterhuene
Copy link
Collaborator

@peterhuene peterhuene commented Oct 9, 2025

This PR implements a walk function that exposes the backends ability to walk a cloud URL for objects prefixed by the given URL.

It can be used to glob a cloud storage URL.

Before submitting this PR, please make sure:

  • You have added a few sentences describing the PR here.
  • You have added yourself or the appropriate individual as the assignee.
  • You have added at least one relevant code reviewer to the PR.
  • Your code builds clean without any errors or warnings.
  • You have added tests (when appropriate).
  • You have updated the README or other documentation to account for these
    changes (when appropriate).
  • You have added an entry to the relevant CHANGELOG.md (see
    "keep a changelog" for more information).
  • Your commit messages follow the conventional commit style.

@peterhuene
Copy link
Collaborator Author

CI failure is due to rust-lang/socket2#616, which hasn't yet made it to a release.

We can ignore the failure for now.

@peterhuene peterhuene force-pushed the walk branch 2 times, most recently from c81fa20 to a71be64 Compare October 9, 2025 18:25
Copy link
Member

@claymcleod claymcleod left a comment

Choose a reason for hiding this comment

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

LGTM

This commit implements a `walk` function that exposes the backends ability to
walk a cloud URL for objects prefixed by the given URL.

It can be used to glob a cloud storage URL.
@peterhuene
Copy link
Collaborator Author

Now we need getrandom to publish containing rust-random/getrandom#732. Going to just merge with the failing docs.

@peterhuene peterhuene merged commit 0788795 into stjude-rust-labs:main Oct 13, 2025
8 of 9 checks passed
@peterhuene peterhuene deleted the walk branch November 7, 2025 16:33
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.

2 participants