Merge pull request #71 from pageuppeople-opensource/feature/SP-389-ad…

…d-retry-policy-when-invoking-lambda bug fix - if succeed, do not repeat :)
pageuppeople-opensource · Sep 8, 2019 · 8fcce0a · 8fcce0a
2 parents 845026f + ef9d4cd
commit 8fcce0a
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/rdl/data_sources/AWSLambdaDataSource.py b/rdl/data_sources/AWSLambdaDataSource.py
@@ -184,9 +184,11 @@ def __invoke_lambda(self, pay_load):
                 # BUT the lambda barfed with an error and therefore the FunctionError would not be None
                 self.logger.error(response_payload)
 
-                if current_attempt >= max_attempts:
+                if current_attempt == max_attempts:
                     raise Exception(
                         "Error received when invoking AWS Lambda. See logs for further details."
                     )
+            else:
+                break
 
         return response_payload