From 2b132a1bdf5994d197a1a7d6755e9dea0d1be32a Mon Sep 17 00:00:00 2001
From: James Kirk <james.kirk@noc.ac.uk>
Date: Tue, 24 Jan 2023 15:20:39 +0000
Subject: [PATCH] refactor: minor tidying up, linking issue, and improving err

---
 rmq.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/rmq.py b/rmq.py
index c8ae7ea..e5f7d35 100644
--- a/rmq.py
+++ b/rmq.py
@@ -2,7 +2,7 @@ import json
 
 import pika
 
-host='localhost' # TODO Handle host being passed in
+host='localhost' # TODO Handle host being passed in (https://git.noc.ac.uk/communications-backbone-system/communications-backbone/-/issues/17)
 
 # -------------------------------------------------------------------------------------------------------------------------------------------------------------
 
@@ -36,7 +36,7 @@ def deliver_to_exchange(channel, body, exchange_name, topic=None):
             )
         )
     else:
-        topic_exchange(channel=channel, exchange_name=exchange_name, topic=topic)
+        topic_exchange(channel=channel, exchange_name=exchange_name)
         channel.basic_publish(
             exchange=exchange_name,
             routing_key=topic, 
@@ -79,7 +79,7 @@ def read_from_queue(queue_name, max_msgs):
             channel.basic_ack(method_frame.delivery_tag)
             try:
                 messages.append(json.loads(body.decode()))
-            except:
+            except json.decoder.JSONDecodeError:
                 messages.append(body.decode())
         else:
             print("No message returned")
@@ -161,7 +161,7 @@ def subscribe(queue_name, exchange_name, topic=None):
         fanout_exchange(channel=channel, exchange_name=exchange_name)
         channel.queue_bind(exchange=exchange_name, queue=queue_name)
     else:
-        topic_exchange(channel=channel, exchange_name=exchange_name, topic=topic)
+        topic_exchange(channel=channel, exchange_name=exchange_name)
         channel.queue_bind(exchange=exchange_name, queue=queue_name, routing_key=topic)
 
     connection.close()
-- 
GitLab