From cf951a820aefcafb51e3e655576a4aa4c29f9e1c Mon Sep 17 00:00:00 2001 From: tcely Date: Sat, 21 Dec 2024 07:53:23 -0500 Subject: [PATCH] response is a dict just now I don't know what I was thinking. Add some type checking while I am at it. --- tubesync/sync/youtube.py | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/tubesync/sync/youtube.py b/tubesync/sync/youtube.py index 2a8de68f..4de2759f 100644 --- a/tubesync/sync/youtube.py +++ b/tubesync/sync/youtube.py @@ -5,7 +5,6 @@ import os -import json from pathlib import Path from django.conf import settings from copy import copy @@ -85,16 +84,14 @@ def _subscriber_only(msg='', response=None): return True else: # ignore msg entirely - try: - data = json.loads(str(response)) - except (TypeError, ValueError, AttributeError): - return False + if not isinstance(response, dict): + raise TypeError(f'response must be a dict, got "{type(response)}" instead') - if 'availability' not in data.keys(): + if 'availability' not in response.keys(): return False # check for the specific expected value - return 'subscriber_only' == data.get('availability') + return 'subscriber_only' == response.get('availability') return False