syncer: fix user-admin-detection, for real

voc · Dec 23, 2024 · 9ae2449 · 9ae2449
1 parent 709d22e
commit 9ae2449
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/syncer.py b/syncer.py
@@ -48,7 +48,12 @@ def asset_to_tiles(asset: Asset):
             }
         )
 
-    if REDIS.get(f"admin:{asset.userid}") != "1":
+    user_is_admin = REDIS.get(f"admin:{asset.userid}")
+    if isinstance(user_is_admin, bytes):
+        # somehow, writing "1" into redis makes it return bytes instead
+        # of a string?
+        user_is_admin = user_is_admin.decode()
+    if user_is_admin != "1":
         tiles.append(
             {
                 "type": "flat",
@@ -71,7 +76,7 @@ def asset_to_tiles(asset: Asset):
                 "config": {
                     "font_size": 25,
                     "fade_time": FADE_TIME,
-                    "text": "Project by @{user} - visit {url} to share your own.".format(
+                    "text": "Project by {user} - visit {url} to share your own.".format(
                         user=asset.username,
                         url=CONFIG["DOMAIN"],
                     ),
@@ -93,7 +98,7 @@ def asset_to_tiles(asset: Asset):
                 asset_states[asset.state] = 0
             asset_states[asset.state] += 1
             log.info(
-                f"asset {asset.id} is in state {asset.state}, uploaded by {asset.user}"
+                f"asset {asset.id} is in state {asset.state}, uploaded by {asset.username}"
             )
 
     msg = []