Merge pull request #120 from stfc/chatops_fixes

BUG: Flask request object has json dict not function

Author: gmatthews20

cloud-chatops/deployments/deployment.yml

@@ -19,7 +19,7 @@ spec:
     spec:
       containers:
       - name: cloud-chatops
-        image: harbor.stfc.ac.uk/stfc-cloud/cloud-chatops:8.0.0
+        image: harbor.stfc.ac.uk/stfc-cloud/cloud-chatops:8.0.1
         env:
           - name: HOST_IP
             valueFrom:

cloud-chatops/deployments/docker-compose.yaml

@@ -1,6 +1,6 @@
 services:
   cloud_chatops:
-    image: harbor.stfc.ac.uk/stfc-cloud/cloud-chatops:8.0.0
+    image: harbor.stfc.ac.uk/stfc-cloud/cloud-chatops:8.0.1
     volumes:
       - /etc/chatops/secrets/:/usr/src/app/cloud_chatops/secrets/
       - /etc/chatops/config/:/usr/src/app/cloud_chatops/config/

cloud-chatops/src/main.py

@@ -71,7 +71,7 @@ def slack_events() -> slack_handler.handle:
 @flask_app.route("/slack/schedule", methods=["POST"])
 def slack_schedule() -> Tuple[str, int]:
     """This function checks the request is authorised then passes it to the weekly reminder calls."""
-    flask_app.logger.info(request.json())
+    flask_app.logger.info(request.json)
     token = request.headers.get("Authorization")
     if token != "token " + get_token("SCHEDULED_REMINDER_TOKEN"):
         flask_app.logger.warning(
@@ -84,7 +84,7 @@ def slack_schedule() -> Tuple[str, int]:
     weekly_reminder(request.json)
     flask_app.logger.info(
         "Request on /slack/schedule by %s executed successfully "
-        "for reminder type %s.", request.remote_addr, request.json()["reminder_type"]
+        "for reminder type %s.", request.remote_addr, request.json["reminder_type"]
     )
     return "OK", 200
 

cloud-chatops/version.txt

@@ -1 +1 @@
-8.0.0
+8.0.1