Skip to content

Add end-to-end filesystem checks and S3 copy compatibility#1

Open
aribon-cs wants to merge 6 commits into
mysamimi:mainfrom
aribon-cs:develop
Open

Add end-to-end filesystem checks and S3 copy compatibility#1
aribon-cs wants to merge 6 commits into
mysamimi:mainfrom
aribon-cs:develop

Conversation

@aribon-cs
Copy link
Copy Markdown

  • Add end-to-end integration coverage for filesystem-backed S3 operations using temporary files
  • Verify payload-based MD5 separately from ETag to avoid incorrect checksum assumptions
  • Extract reusable app/bootstrap wiring into src/lib.rs so integration tests can boot the real server
  • Implement S3-compatible CopyObject handling via x-amz-copy-source
  • Add minimal GET ?acl compatibility to support Flysystem/AWS S3 copy flows
  • Update integration tests to exercise real copy semantics instead of emulated read/write copy

Why

  • Flysystem checksum defaults to ETag, while FerroS3 ETag is not a content MD5
  • Flysystem rename/move relies on S3 copy semantics, which FerroS3 did not previously support
  • The previous copy test validated a helper path, not the actual S3 adapter behavior

Validation

    • cargo test

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