splitlog 4.1.1__tar.gz → 4.1.3__tar.gz

{splitlog-4.1.1 → splitlog-4.1.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: splitlog
-Version: 4.1.1
+Version: 4.1.3
 Summary: Utility to split aggregated logs from Apache Hadoop Yarn applications into a folder hierarchy
 License-Expression: MIT
 License-File: LICENSE

{splitlog-4.1.1 → splitlog-4.1.3}/pyproject.toml

@@ -26,7 +26,7 @@ dependencies = [
     "python-dateutil (>=2.9.0,<3.0.0)",
     "pytz (>=2025.2)",
 ]
-version = "4.1.1"
+version = "4.1.3"
 
 
 [project.urls]

{splitlog-4.1.1 → splitlog-4.1.3}/splitlog/outputfolder.py

@@ -77,7 +77,7 @@ def _is_relative_to(path: Path, parent: Path) -> bool:
         return True
 
 
-class DefaultLocalFilesystemOutputFolder(OutputFolder, metaclass=abc.ABCMeta):
+class DefaultLocalFilesystemOutputFolder(OutputFolder):
     """Encapsulates filesystem IO on output folders.
 
     This implementation is portable but unsafe to use when invoking splitlog with privileges.
@@ -114,11 +114,11 @@ class DefaultLocalFilesystemOutputFolder(OutputFolder, metaclass=abc.ABCMeta):
         os.mkdir(real_path, mode=self.DIR_MODE)
 
     def _check_paths(self, path: Path) -> Path:
-        assert not path.is_absolute(), f"Path {path} must be relative"
+        if path.is_absolute():
+            raise ValueError(f"Path {path} must be relative")
         real_path = Path(os.path.normpath(self._path / path))
-        assert _is_relative_to(
-            real_path, self._path
-        ), f"Path {path} outside {self._path}"
+        if not _is_relative_to(real_path, self._path):
+            raise ValueError(f"Path {path} outside {self._path}")
         return real_path
 
     def create(self, path: Path) -> BinWriter:
@@ -214,18 +214,18 @@ class LinuxLocalFilesystemOutputFolder(OutputFolder):
 
     def create(self, path: Path) -> BinWriter:
         real_path = self._ensure_path_under_root(path)
-        return FileWrapper(open(path, "xb", opener=self._opener))
+        return FileWrapper(open(real_path, "xb", opener=self._opener))
 
     def _ensure_path_under_root(self, path: Path) -> Path:
-        assert not path.is_absolute(), f"Path {path} must be relative"
+        if path.is_absolute():
+            raise ValueError(f"Path {path} must be relative")
 
         # remove all ".." and "." components
         real_path = Path(os.path.normpath(self._path / path))
 
-        # assert resulting absolute path is still inside self._path
-        assert _is_relative_to(
-            real_path, self._path
-        ), f"Path {path} outside {self._path}"
+        # ensure resulting absolute path is still inside self._path
+        if not _is_relative_to(real_path, self._path):
+            raise ValueError(f"Path {path} outside {self._path}")
         return real_path.relative_to(self._path)
 
     @staticmethod