Merge pull request #56 from wp-media/develop

Enhance logs
wp-media · Nov 9, 2023 · 81b30fd · 81b30fd
2 parents 8f823d9 + f2b4ad3
commit 81b30fd
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/sources/handlers/DeployHandler.py b/sources/handlers/DeployHandler.py
@@ -40,8 +40,8 @@ def deploy_commit(self, app_context, task_params: DeployHandlerParam):
         request_payload['environment'] = task_params.env
         request_payload['ref'] = task_params.commit
 
-        current_app.logger.info("deploy_commit: Requesting a deployment:/n" +
-                                json.dumps(request_payload) + "/n" +
+        current_app.logger.info("deploy_commit: Requesting a deployment:\n" +
+                                json.dumps(request_payload) + "\n" +
                                 json.dumps(request_header))
         result = requests.post(url=self.godp_deploy_url,
                                headers=request_header,
@@ -50,8 +50,4 @@ def deploy_commit(self, app_context, task_params: DeployHandlerParam):
             current_app.logger.error("deploy_commit: GODP call failed.")
             raise ValueError('GODP call failed.')
         result_json = result.json()
-        if result_json["url"]:
-            current_app.logger.info("deploy_commit: GODP Response:/n" + result_json)
-        else:
-            current_app.logger.error("deploy_commit: GODP did not return a deploy URL.")
-            raise ValueError('GODP did not return a deploy URL.')
+        current_app.logger.info("deploy_commit: GODP Response:\n" + result_json)