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.

Sign up for GitHub

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

Jump to bottom

Fix S3 destroy for feeds with sub paths #188

Merged

emgarten merged 1 commit into main from emgarten/subfeed-destroy

Feb 3, 2024

Owner

emgarten commented Feb 3, 2024 •

edited

Loading

Updated AmazonS3FileSystem.GetFiles to account for sub paths when retrieving all files in the feed
- This call is currently only used by Destroy, other scenarios should not be impacted
- Previous behavior was resulting in the sub path being duplicated in the URI, which resulted in files not being found.
Updated the S3 sub path functional test to verify the feed is fully removed.

Fixes #185


          Update index.md

3a3989f

emgarten force-pushed the emgarten/subfeed-destroy branch from 9051989 to 3a3989f Compare

February 3, 2024 02:50

emgarten merged commit 102c3ac into main

5 checks passed

emgarten deleted the emgarten/subfeed-destroy branch

February 3, 2024 03:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet