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 issue with S3LayerDeleter deleting files outside of layer #2070

Merged
merged 1 commit into from Mar 14, 2017

Conversation

Projects
None yet
2 participants
@kthompson
Contributor

kthompson commented Mar 14, 2017

S3LayerDeleter can incorrectly delete keys since it is based on a wildcard pattern.

For example:
If you have multiple keys like

1/files
10/files

and you delete an entry with the key 1 then it will also delete 10/files Appending the extra / will perform the task as expected

@lossyrob lossyrob added this to the 1.1 milestone Mar 14, 2017

Update S3LayerDeleter.scala
S3LayerDeleter can incorrectly delete keys since it is based on a wildcard pattern.

@lossyrob lossyrob changed the title from Update S3LayerDeleter.scala to Fix issue with S3LayerDeleter deleting files outside of layer Mar 14, 2017

@lossyrob lossyrob merged commit df7c945 into locationtech:master Mar 14, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@kthompson kthompson deleted the kthompson:patch-1 branch Mar 20, 2017

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