Skip to content

fix(cells): harden Ktor S3 transfers PR 2 - #4348

Draft
MohamadJaara wants to merge 6 commits into
mo/cells-s3-kmp-clientfrom
mo/cells-s3-kmp-client-hardening
Draft

fix(cells): harden Ktor S3 transfers PR 2#4348
MohamadJaara wants to merge 6 commits into
mo/cells-s3-kmp-clientfrom
mo/cells-s3-kmp-client-hardening

Conversation

@MohamadJaara

Copy link
Copy Markdown
Member

Stack

Intent

Keep the shared KMP S3 migration review focused by moving defensive response handling and expanded edge-case coverage into a follow-up layer.

Changes

  • Add strict namespace-aware parsing for S3 XML responses and embedded errors.
  • Validate streamed download lengths without replaying partial destinations.
  • Use a typed failure when Cells credentials are unavailable.
  • Expand multipart, signing, retry, cancellation, and malformed-response coverage.
  • Preserve the existing per-request token refresh, retry delay, and multipart cleanup behavior.

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Test Results

0 tests   - 4 962   0 ✅  - 4 942   0s ⏱️ - 8m 39s
0 suites  -    13   0 💤  -    20 
0 files    -    13   0 ❌ ±    0 

Results for commit 0d0bd18. ± Comparison against base commit 604f249.

♻️ This comment has been updated with latest results.

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