chore: fix exception handling here as well

This commit is contained in:
2025-09-11 14:56:58 +02:00
parent 294f73409f
commit 8121d18eb4

View File

@@ -22,11 +22,18 @@ def get_topics_by_id(topic_id):
Returns: Returns:
dict or None: The topic data if successful, otherwise None. dict or None: The topic data if successful, otherwise None.
""" """
response = requests.get(f"{DISCOURSE_BASE_URL}/t/{topic_id}.json", headers=headers) try:
response = requests.get(
f"{DISCOURSE_BASE_URL}/t/{topic_id}.json",
headers=headers,
timeout=10, # prevent hanging forever
)
if response.status_code == 200: if response.status_code == 200:
return response.json() return response.json()
elif response.status_code == 403: elif response.status_code == 403:
logger.error(f"Access forbidden for topic {topic_id}: {response.status_code}") logger.error(
f"Access forbidden for topic {topic_id}: {response.status_code}"
)
return None return None
else: else:
logger.error( logger.error(
@@ -34,6 +41,10 @@ def get_topics_by_id(topic_id):
) )
return None return None
except requests.exceptions.RequestException as e:
logger.error(f"Request failed for topic {topic_id}: {e}")
return None
def get_topics_by_tag(tag_name): def get_topics_by_tag(tag_name):
""" """