Files.com 1.6.528__tar.gz → 1.6.529__tar.gz

{files_com-1.6.528/files_com.egg-info → files_com-1.6.529}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: files_com
-Version: 1.6.528
+Version: 1.6.529
 Summary: Python bindings for the Files.com API
 License: MIT
 Requires-Python: >=3.5

files_com-1.6.529/_VERSION

@@ -0,0 +1 @@
+1.6.529

{files_com-1.6.528 → files_com-1.6.529/files_com.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: files_com
-Version: 1.6.528
+Version: 1.6.529
 Summary: Python bindings for the Files.com API
 License: MIT
 Requires-Python: >=3.5

{files_com-1.6.528 → files_com-1.6.529}/files_sdk/__init__.py

@@ -320,7 +320,7 @@ source_ip = None
 workspace_id = None
 base_url = "https://app.files.com"
 base_path = "api/rest/v1"
-version = "1.6.528"
+version = "1.6.529"
 
 __version__ = version
 

{files_com-1.6.528 → files_com-1.6.529}/files_sdk/models/file.py

@@ -1448,6 +1448,42 @@ def begin_upload(path, params=None, options=None):
     ]
 
 
+_WINDOWS_DEVICE_NAMES = frozenset(
+    [
+        "CON",
+        "PRN",
+        "AUX",
+        "NUL",
+        "CONIN$",
+        "CONOUT$",
+        "COM1",
+        "COM2",
+        "COM3",
+        "COM4",
+        "COM5",
+        "COM6",
+        "COM7",
+        "COM8",
+        "COM9",
+        "COM\u00b9",
+        "COM\u00b2",
+        "COM\u00b3",
+        "LPT1",
+        "LPT2",
+        "LPT3",
+        "LPT4",
+        "LPT5",
+        "LPT6",
+        "LPT7",
+        "LPT8",
+        "LPT9",
+        "LPT\u00b9",
+        "LPT\u00b2",
+        "LPT\u00b3",
+    ]
+)
+
+
 def _underscore_destination_path(root, destination_id, relative_path=None):
     return path_util.normalize(
         f"_/{root}/{destination_id}/{relative_path or ''}"
@@ -1644,11 +1680,23 @@ def move_to_child_site(path, site_id, destination, params=None, options=None):
     return move(path, params, options)
 
 
+def _is_windows_device_name(name):
+    # A bare device name is always a device. Anything after it, such as the
+    # extension in NUL.txt, makes it a device on some Windows versions and a
+    # file on others, so Windows decides those names: it resolves a device to
+    # a \\.\ path.
+    base = name.split(".", 1)[0].rstrip(" ")
+    if base.upper() not in _WINDOWS_DEVICE_NAMES:
+        return False
+    return base == name or os.path.abspath(name).startswith("\\\\.\\")
+
+
 def _default_local_path(path):
     name = path_util.normalize(path).split("/")[-1]
     # On Windows a colon makes the name a drive-relative path or an NTFS
-    # stream, which can write outside the working directory or onto another file.
-    if os.name == "nt" and ":" in name:
+    # stream, which can write outside the working directory or onto another
+    # file, and a device name such as NUL writes to the device instead of a file.
+    if os.name == "nt" and (":" in name or _is_windows_device_name(name)):
         raise InvalidParameterError(
             f"Bad parameter: local_path is required because {name!r} is not a valid local file name"
         )

{files_com-1.6.528 → files_com-1.6.529}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "files_com"
-version = "1.6.528"
+version = "1.6.529"
 description = "Python bindings for the Files.com API"
 readme = "README.md"
 license = { text = "MIT" }

{files_com-1.6.528 → files_com-1.6.529}/tests/test_file_download_default_path.py

@@ -13,8 +13,8 @@ class TestFileDownloadDefaultPath(unittest.TestCase):
 
         mock_download_file.assert_called_once_with("report:2026.txt")
 
-    def test_windows_download_without_local_path_rejects_drive_and_stream_names(self):
-        for remote_path in ("folder/C:victim.txt", "C:victim.txt", "folder/victim.txt::$DATA"):
+    def test_windows_download_without_local_path_rejects_drive_stream_and_device_names(self):
+        for remote_path in ("folder/C:victim.txt", "C:victim.txt", "folder/victim.txt::$DATA", "folder/NUL", "folder/con", "COM1", "folder/CONIN$"):
             with self.subTest(remote_path=remote_path):
                 with mock.patch.object(file_model.os, "name", "nt"), mock.patch.object(file_model.File, "download_file") as mock_download_file:
                     with self.assertRaises(InvalidParameterError):

files_com-1.6.528/_VERSION

@@ -1 +0,0 @@
-1.6.528