diff options
| author | Benjamin Copeland <ben.copeland@linaro.org> | 2018-11-15 15:59:36 +0000 |
|---|---|---|
| committer | Benjamin Copeland <ben.copeland@linaro.org> | 2018-11-16 14:35:06 +0000 |
| commit | 11f14aac5a0c437e0ad345d6b9b2309c5fd0704e (patch) | |
| tree | a068bf13b369688d9af1c0c217c2ad6840c20214 /3336e46 | |
| parent | b156abd87b29c82da267d8030a8409531d3cc5f1 (diff) | |
s3_purge: fix the releases deletemarker bug and code cleanup
When testing this the releases/ bucket wasn't checking that the
deletemarker was being set. Obviously this is very dangerous so a fix
has been put in to ensure a deletemarker *is* set before deleting the
file.
Since we handle the same call a couple of times, I have put
handle_permanent_deletes, into its own function which has reduced a lot
of reputation.
Along with those fix, the code has been cleaned up to be a bit more
readable
Change-Id: Iffd2f55d04bc889050bfd7a19e6f62fe205de0c6
Reviewed-on: https://review.linaro.org/29085
Reviewed-by: Benjamin Copeland <ben.copeland@linaro.org>
Diffstat (limited to '3336e46')
0 files changed, 0 insertions, 0 deletions