Continued:

- changed string "true/false" to boolean
- please update your configuration file (all occurances!)

diff --git a/config.defaults.json b/config.defaults.json
index 8b728462d490af4b2be0a8f1671ec31de1e9c2cf..a616c72d6e67113c03fe742809561fbf57ac5da7 100644 (file)
--- a/config.defaults.json
+++ b/config.defaults.json
@@ -22,11 +22,11 @@
     "source_last_access"      : 86400,
     "misskey_limit"           : 100,
     "error_log_cleanup"       : 604800,
-    "write_error_log"         : "true",
+    "write_error_log"         : true,
     "rss_limit"               : 50,
     "api_limit"               : 500,
     "theme"                   : "light",
-    "allow_i2p_domain"        : "false",
+    "allow_i2p_domain"        : false,
     "instances_social_api_key": "",
     "max_crawl_depth"         : 2000,
     "min_peers_length"        : 1000

diff --git a/fba/commands.py b/fba/commands.py
index ce6623e7c0336bb3219f581e14532fa8509ab149..ef9d746373d442bd54d7bcdf0406b167b69798fd 100644 (file)
--- a/fba/commands.py
+++ b/fba/commands.py
@@ -373,7 +373,7 @@ def fetch_blocks(args: argparse.Namespace) -> int:
             elif block["blocked"].endswith(".onion"):
                 logger.debug("blocked='%s' is a TOR .onion domain - SKIPPED", block["blocked"])
                 continue
-            elif block["blocked"].endswith(".i2p") and not config.get("allow_i2p_domain") == "true":
+            elif block["blocked"].endswith(".i2p") and not config.get("allow_i2p_domain"):
                 logger.debug("blocked='%s' is an I2P .onion domain - SKIPPED", block["blocked"])
                 continue
             elif block["blocked"].endswith(".arpa"):
@@ -1342,7 +1342,7 @@ def recheck_obfuscation(args: argparse.Namespace) -> int:
             elif block["blocked"].endswith(".onion"):
                 logger.debug("blocked='%s' is a TOR onion domain name - SKIPPED!", block["blocked"])
                 continue
-            elif block["blocked"].endswith(".i2p") and not config.get("allow_i2p_domain") == "true":
+            elif block["blocked"].endswith(".i2p") and not config.get("allow_i2p_domain"):
                 logger.debug("blocked='%s' is an I2P onion domain name - SKIPPED!", block["blocked"])
                 continue
             elif block["blocked"].endswith(".arpa"):

diff --git a/fba/helpers/domain.py b/fba/helpers/domain.py
index 86148928e405748613a914f241f93489aed216b5..df4c70398d60b0a390eeabcaaaeeb1c48aff2455 100644 (file)
--- a/fba/helpers/domain.py
+++ b/fba/helpers/domain.py
@@ -42,7 +42,7 @@ def raise_on(domain: str):
         raise ValueError(f"domain='{domain}' is not a valid domain")
     elif domain.endswith(".onion"):
         raise ValueError(f"domain='{domain}' is a TOR, please don't crawl them!")
-    elif domain.endswith(".i2p") and not config.get("allow_i2p_domain") == "true":
+    elif domain.endswith(".i2p") and not config.get("allow_i2p_domain"):
         raise ValueError(f"domain='{domain}' is an I2P, please don't crawl them!")
     elif domain.endswith(".arpa"):
         raise ValueError(f"domain='{domain}' is a domain for reversed IP addresses, please don't crawl them!")
@@ -97,7 +97,7 @@ def is_wanted(domain: str) -> bool:
     elif domain.endswith(".onion"):
         logger.debug("domain='%s' is a TOR .onion domain - setting False ...", domain)
         wanted = False
-    elif domain.endswith(".i2p") and not config.get("allow_i2p_domain") == "true":
+    elif domain.endswith(".i2p") and not config.get("allow_i2p_domain"):
         logger.debug("domain='%s' is an I2P domain - setting False ...", domain)
         wanted = False
     elif domain.endswith(".tld"):

diff --git a/fba/helpers/processing.py b/fba/helpers/processing.py
index 5a96ac9bf22b9476ccc9764203670e3d1346e520..847f9fca2fed7bd9992c931b3beab0973a91434a 100644 (file)
--- a/fba/helpers/processing.py
+++ b/fba/helpers/processing.py
@@ -205,7 +205,7 @@ def csv_block(blocker: str, url: str, command: str):
         elif domain.endswith(".onion"):
             logger.debug("domain='%s' is a TOR .onion domain - SKIPPED", domain)
             continue
-        elif domain.endswith(".i2p") and not config.get("allow_i2p_domain") == "true":
+        elif domain.endswith(".i2p") and not config.get("allow_i2p_domain"):
             logger.debug("domain='%s' is an I2P .onion domain - SKIPPED", domain)
             continue
         elif domain.endswith(".arpa"):

diff --git a/fba/models/error_log.py b/fba/models/error_log.py
index 35e0933c3a514fcec619657ed3f34af2f230ddf9..672c775693e2ed49a99b199263ca66cbd8443491 100644 (file)
--- a/fba/models/error_log.py
+++ b/fba/models/error_log.py
@@ -32,7 +32,7 @@ def add(domain: str, error: dict):
 
     if blacklist.is_blacklisted(domain):
         raise ValueError(f"domain='{domain}' is blacklisted but function was invoked")
-    elif config.get("write_error_log").lower() != "true":
+    elif not config.get("write_error_log"):
         logger.debug("Writing to error_log is disabled in configuruation file - EXIT!")
         return