airbnb / binaryalert

BinaryAlert: Serverless, Real-time & Retroactive Malware Detection.
https://binaryalert.io
Apache License 2.0
1.41k stars 187 forks source link

live_test remove DynamoDB only on success #39

Closed austinbyers closed 7 years ago

austinbyers commented 7 years ago

This commit was supposed to be in the last PR (#36), I accidentally merged too soon.

If the live_test fails, then by definition the Dynamo entry is not present, so there is no need to try to remove it.

Tested

Deploy in test account and python3 manage.py live_test

to: @jacknagz or @mime-frame cc: @airbnb/binaryalert-maintainers