From 6a2a2b5c3da79fe0097c98a04e435e493c73223d Mon Sep 17 00:00:00 2001
From: Numerlor <25886452+Numerlor@users.noreply.github.com>
Date: Wed, 26 Feb 2020 00:29:52 +0100
Subject: Check for empty strings alongside None before loading cog.

Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.
---
 bot/cogs/reddit.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/bot/cogs/reddit.py b/bot/cogs/reddit.py
index 3278363ba..7cb340145 100644
--- a/bot/cogs/reddit.py
+++ b/bot/cogs/reddit.py
@@ -290,7 +290,8 @@ class Reddit(Cog):
 
 def setup(bot: Bot) -> None:
     """Load the Reddit cog."""
-    if None in (RedditConfig.client_id, RedditConfig.secret):
+    invalid_values = "", None
+    if any(value in (RedditConfig.secret, RedditConfig.client_id) for value in invalid_values):
         log.error("Credentials not provided, cog not loaded.")
         return
     bot.add_cog(Reddit(bot))
-- 
cgit v1.2.3