Skip to content

feat(volcengine-tos): support presigned URLs#779

Open
ddupg wants to merge 4 commits into
apache:mainfrom
ddupg:codex/ddu-27-volcengine-tos-presign
Open

feat(volcengine-tos): support presigned URLs#779
ddupg wants to merge 4 commits into
apache:mainfrom
ddupg:codex/ddu-27-volcengine-tos-presign

Conversation

@ddupg

@ddupg ddupg commented Jul 10, 2026

Copy link
Copy Markdown
Member

Part of #752.

Summary

  • Add query-based TOS signing for presigned URLs.
  • Support security tokens, signed TOS headers, and explicit payload hashes.
  • Add deterministic coverage and a credential-gated real TOS integration test.

Changes outside the TOS service only fix Rust 1.97 Clippy warnings exposed by CI.

Verification

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --no-fail-fast
  • Real TOS presigned PUT/GET/DELETE
  • OpenDAL TOS behavior suite: 54 passed on a versioning-enabled bucket

@ddupg ddupg force-pushed the codex/ddu-27-volcengine-tos-presign branch from 1b8eb5c to 4d25277 Compare July 10, 2026 08:42
@ddupg ddupg marked this pull request as ready for review July 10, 2026 10:06
@ddupg

ddupg commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

@Xuanwo Could you help review this? thanks!

ddupg added 4 commits July 13, 2026 20:36
Change-Id: I49dbab0602b70db4b46791fa71ce12479f9608fb
Change-Id: I4110f5318d89a734be623853aacae0a470de455e
Change-Id: I1fc962116bdfe6934eddd73f03ab56cc7e9a554b
Change-Id: I74162a4a724a2120913f73b8b81d97a9016597ba
@ddupg ddupg force-pushed the codex/ddu-27-volcengine-tos-presign branch from c25e760 to 757e77a Compare July 13, 2026 12:37
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.

1 participant