Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Properly handle nested directories in S3 #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

charmitro
Copy link

This fixes the issue with nested directories not being properly handled when using rclone with the S3 server. The fix implements a more accurate approach to directory listings that:

  1. Correctly handles the hierarchical structure of directories
  2. Properly applies S3 delimiter logic for nested paths
  3. Ensures files in nested directories are correctly reported
  4. Makes recursive copying work as expected

Added a test to verify recursive copy functionality.

This fixes the issue with nested directories not being properly handled when using rclone with the S3 server. The fix implements a more accurate approach to directory listings that:

1. Correctly handles the hierarchical structure of directories
2. Properly applies S3 delimiter logic for nested paths
3. Ensures files in nested directories are correctly reported
4. Makes recursive copying work as expected

Added a test to verify recursive copy functionality.
@charmitro charmitro force-pushed the fix-recursive-copies branch from cc84940 to 47136e2 Compare March 19, 2025 11:56
@charmitro charmitro force-pushed the fix-recursive-copies branch from 47136e2 to ae583b0 Compare March 19, 2025 12:09
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.

None yet

1 participant