refac

Author: taylorwilsdon

core/utils.py

@@ -32,15 +32,15 @@ class UserInputError(Exception):
 
 # Directories from which local file reads are allowed.
 # The user's home directory is the default safe base.
-# Override via ALLOWED_FILE_DIRS env var (colon-separated paths).
+# Override via ALLOWED_FILE_DIRS env var (os.pathsep-separated paths).
 _ALLOWED_FILE_DIRS_ENV = "ALLOWED_FILE_DIRS"
 
 
 def _get_allowed_file_dirs() -> list[Path]:
     """Return the list of directories from which local file access is permitted."""
     env_val = os.environ.get(_ALLOWED_FILE_DIRS_ENV)
     if env_val:
-        return [Path(p).resolve() for p in env_val.split(":") if p.strip()]
+        return [Path(p).expanduser().resolve() for p in env_val.split(os.pathsep) if p.strip()]
     home = Path.home()
     return [home] if home else []
 

gdrive/drive_tools.py

@@ -733,14 +733,7 @@ def _resolve_and_validate_host(hostname: str) -> list[str]:
     for _family, _type, _proto, _canonname, sockaddr in addr_infos:
         ip_str = sockaddr[0]
         ip = ipaddress.ip_address(ip_str)
-        if (
-            ip.is_private
-            or ip.is_loopback
-            or ip.is_reserved
-            or ip.is_link_local
-            or ip.is_multicast
-            or ip.is_unspecified
-        ):
+        if not ip.is_global:
             raise ValueError(
                 f"URLs pointing to private/internal networks are not allowed: "
                 f"{hostname} resolves to {ip_str}"