dar-backup 0.8.0__tar.gz → 0.8.2__tar.gz

{dar_backup-0.8.0/src/dar_backup → dar_backup-0.8.2}/Changelog.md

@@ -1,6 +1,32 @@
 <!-- markdownlint-disable MD024 -->
 # dar-backup Changelog
 
+## v2-beta-0.8.2 - 2025-07-17
+
+Github link: [v2-beta-0.8.2](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.2/v2)
+
+### Added
+
+- Security hardening: CommandRunner now performs strict command-line sanitization
+  - Disallows potentially dangerous characters (e.g. ;, &, |) in command arguments
+  - Prevents injection-style misuse when restoring specific files or invoking custom commands
+
+- Documentation:
+  - New [README section](https://github.com/per2jensen/dar-backup?tab=readme-ov-file#limitations-on-file-names-with-special-characters) explains filename restrictions and safe workarounds (e.g. restoring directly with dar, if needed)
+  - Includes a Markdown table listing all disallowed characters
+
+- Test suite:
+  - Existing test cases updated to comply with the new sanitization rules
+  - Additional tests ensure CommandRunner handles large binary output and edge cases cleanly
+
+## v2-beta-0.8.1 - 2025-07-16
+
+Github link: [v2-beta-0.8.1](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.1/v2)
+
+### Added
+
+- FIX: runner now logs an error and fills more data into the returned CommandResult.
+
 ## v2-beta-0.8.0 - 2025-06-13
 
 Github link: [v2-beta-0.8.0](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.0/v2)

{dar_backup-0.8.0 → dar_backup-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dar-backup
-Version: 0.8.0
+Version: 0.8.2
 Summary: A script to do full, differential and incremental backups using dar.     Some files are restored from the backups during verification, after which par2 redundancy files are created.     The script also has a cleanup feature to remove old backups and par2 files.
 Project-URL: GPG Public Key, https://keys.openpgp.org/search?q=dar-backup@pm.me
 Project-URL: Homepage, https://github.com/per2jensen/dar-backup/tree/main/v2
@@ -727,7 +727,7 @@ Description-Content-Type: text/markdown
 [![PyPI version](https://img.shields.io/pypi/v/dar-backup.svg)](https://pypi.org/project/dar-backup/)
 [![PyPI downloads](https://img.shields.io/badge/dynamic/json?color=blue&label=PyPI%20downloads&query=total&url=https%3A%2F%2Fraw.githubusercontent.com%2Fper2jensen%2Fdar-backup%2Fmain%2Fdownloads.json)](https://pypi.org/project/dar-backup/)
 [![# clones](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/badge_clones.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
-[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
+[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)  <sub>🎯 Stats powered by [ClonePulse](https://github.com/per2jensen/clonepulse)</sub>
 
 The wonderful 'dar' [Disk Archiver](https://github.com/Edrusb/DAR) is used for
 the heavy lifting, together with the [parchive](https://github.com/Parchive/par2cmdline) suite in these scripts.

{dar_backup-0.8.0 → dar_backup-0.8.2}/README.md

@@ -9,7 +9,7 @@
 [![PyPI version](https://img.shields.io/pypi/v/dar-backup.svg)](https://pypi.org/project/dar-backup/)
 [![PyPI downloads](https://img.shields.io/badge/dynamic/json?color=blue&label=PyPI%20downloads&query=total&url=https%3A%2F%2Fraw.githubusercontent.com%2Fper2jensen%2Fdar-backup%2Fmain%2Fdownloads.json)](https://pypi.org/project/dar-backup/)
 [![# clones](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/badge_clones.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
-[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
+[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)  <sub>🎯 Stats powered by [ClonePulse](https://github.com/per2jensen/clonepulse)</sub>
 
 The wonderful 'dar' [Disk Archiver](https://github.com/Edrusb/DAR) is used for
 the heavy lifting, together with the [parchive](https://github.com/Parchive/par2cmdline) suite in these scripts.

{dar_backup-0.8.0 → dar_backup-0.8.2}/doc/badges/badge_clones.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
   "label": "# clones",
-  "message": "1012",
+  "message": "1475",
   "color": "deeppink"
 }

{dar_backup-0.8.0 → dar_backup-0.8.2}/doc/clones.json

@@ -13,8 +13,8 @@
       "label": "Daily max: 124"
     }
   ],
-  "total_clones": 1012,
-  "unique_clones": 513,
+  "total_clones": 1475,
+  "unique_clones": 838,
   "daily": [
     {
       "timestamp": "2025-05-04T00:00:00Z",
@@ -153,68 +153,238 @@
     },
     {
       "timestamp": "2025-05-31T00:00:00Z",
-      "count": 13,
-      "uniques": 7
+      "count": 8,
+      "uniques": 5
     },
     {
       "timestamp": "2025-06-01T00:00:00Z",
-      "count": 31,
-      "uniques": 17
+      "count": 25,
+      "uniques": 15
     },
     {
       "timestamp": "2025-06-02T00:00:00Z",
-      "count": 21,
-      "uniques": 10
+      "count": 15,
+      "uniques": 7
     },
     {
       "timestamp": "2025-06-03T00:00:00Z",
-      "count": 62,
-      "uniques": 33
+      "count": 57,
+      "uniques": 31
     },
     {
       "timestamp": "2025-06-04T00:00:00Z",
-      "count": 45,
-      "uniques": 26
+      "count": 35,
+      "uniques": 22
     },
     {
       "timestamp": "2025-06-05T00:00:00Z",
-      "count": 24,
-      "uniques": 17
+      "count": 15,
+      "uniques": 12
     },
     {
       "timestamp": "2025-06-06T00:00:00Z",
-      "count": 45,
-      "uniques": 23
+      "count": 33,
+      "uniques": 18
     },
     {
       "timestamp": "2025-06-07T00:00:00Z",
-      "count": 60,
-      "uniques": 25
+      "count": 49,
+      "uniques": 20
     },
     {
       "timestamp": "2025-06-08T00:00:00Z",
-      "count": 43,
-      "uniques": 23
+      "count": 33,
+      "uniques": 19
     },
     {
       "timestamp": "2025-06-09T00:00:00Z",
-      "count": 28,
-      "uniques": 17
+      "count": 18,
+      "uniques": 12
     },
     {
       "timestamp": "2025-06-10T00:00:00Z",
-      "count": 34,
-      "uniques": 21
+      "count": 26,
+      "uniques": 16
     },
     {
       "timestamp": "2025-06-11T00:00:00Z",
+      "count": 9,
+      "uniques": 6
+    },
+    {
+      "timestamp": "2025-06-12T00:00:00Z",
+      "count": 10,
+      "uniques": 8
+    },
+    {
+      "timestamp": "2025-06-13T00:00:00Z",
+      "count": 48,
+      "uniques": 18
+    },
+    {
+      "timestamp": "2025-06-14T00:00:00Z",
+      "count": 11,
+      "uniques": 9
+    },
+    {
+      "timestamp": "2025-06-15T00:00:00Z",
+      "count": 10,
+      "uniques": 8
+    },
+    {
+      "timestamp": "2025-06-16T00:00:00Z",
+      "count": 18,
+      "uniques": 11
+    },
+    {
+      "timestamp": "2025-06-17T00:00:00Z",
+      "count": 11,
+      "uniques": 9
+    },
+    {
+      "timestamp": "2025-06-18T00:00:00Z",
+      "count": 9,
+      "uniques": 7
+    },
+    {
+      "timestamp": "2025-06-19T00:00:00Z",
+      "count": 10,
+      "uniques": 8
+    },
+    {
+      "timestamp": "2025-06-20T00:00:00Z",
+      "count": 25,
+      "uniques": 18
+    },
+    {
+      "timestamp": "2025-06-21T00:00:00Z",
+      "count": 8,
+      "uniques": 6
+    },
+    {
+      "timestamp": "2025-06-22T00:00:00Z",
+      "count": 8,
+      "uniques": 6
+    },
+    {
+      "timestamp": "2025-06-23T00:00:00Z",
+      "count": 19,
+      "uniques": 14
+    },
+    {
+      "timestamp": "2025-06-24T00:00:00Z",
+      "count": 8,
+      "uniques": 6
+    },
+    {
+      "timestamp": "2025-06-25T00:00:00Z",
+      "count": 11,
+      "uniques": 8
+    },
+    {
+      "timestamp": "2025-06-26T00:00:00Z",
+      "count": 6,
+      "uniques": 5
+    },
+    {
+      "timestamp": "2025-06-27T00:00:00Z",
+      "count": 11,
+      "uniques": 7
+    },
+    {
+      "timestamp": "2025-06-28T00:00:00Z",
+      "count": 12,
+      "uniques": 9
+    },
+    {
+      "timestamp": "2025-06-29T00:00:00Z",
+      "count": 10,
+      "uniques": 7
+    },
+    {
+      "timestamp": "2025-06-30T00:00:00Z",
       "count": 20,
       "uniques": 13
     },
     {
-      "timestamp": "2025-06-12T00:00:00Z",
+      "timestamp": "2025-07-01T00:00:00Z",
+      "count": 12,
+      "uniques": 10
+    },
+    {
+      "timestamp": "2025-07-02T00:00:00Z",
+      "count": 12,
+      "uniques": 10
+    },
+    {
+      "timestamp": "2025-07-03T00:00:00Z",
+      "count": 12,
+      "uniques": 10
+    },
+    {
+      "timestamp": "2025-07-04T00:00:00Z",
       "count": 18,
+      "uniques": 9
+    },
+    {
+      "timestamp": "2025-07-05T00:00:00Z",
+      "count": 21,
+      "uniques": 14
+    },
+    {
+      "timestamp": "2025-07-06T00:00:00Z",
+      "count": 20,
+      "uniques": 15
+    },
+    {
+      "timestamp": "2025-07-07T00:00:00Z",
+      "count": 26,
+      "uniques": 17
+    },
+    {
+      "timestamp": "2025-07-08T00:00:00Z",
+      "count": 14,
+      "uniques": 9
+    },
+    {
+      "timestamp": "2025-07-09T00:00:00Z",
+      "count": 19,
+      "uniques": 13
+    },
+    {
+      "timestamp": "2025-07-10T00:00:00Z",
+      "count": 18,
+      "uniques": 13
+    },
+    {
+      "timestamp": "2025-07-11T00:00:00Z",
+      "count": 22,
+      "uniques": 13
+    },
+    {
+      "timestamp": "2025-07-12T00:00:00Z",
+      "count": 19,
       "uniques": 12
+    },
+    {
+      "timestamp": "2025-07-13T00:00:00Z",
+      "count": 20,
+      "uniques": 13
+    },
+    {
+      "timestamp": "2025-07-14T00:00:00Z",
+      "count": 38,
+      "uniques": 21
+    },
+    {
+      "timestamp": "2025-07-15T00:00:00Z",
+      "count": 18,
+      "uniques": 12
+    },
+    {
+      "timestamp": "2025-07-16T00:00:00Z",
+      "count": 30,
+      "uniques": 18
     }
   ]
 }

{dar_backup-0.8.0 → dar_backup-0.8.2/src/dar_backup}/Changelog.md

@@ -1,6 +1,32 @@
 <!-- markdownlint-disable MD024 -->
 # dar-backup Changelog
 
+## v2-beta-0.8.2 - 2025-07-17
+
+Github link: [v2-beta-0.8.2](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.2/v2)
+
+### Added
+
+- Security hardening: CommandRunner now performs strict command-line sanitization
+  - Disallows potentially dangerous characters (e.g. ;, &, |) in command arguments
+  - Prevents injection-style misuse when restoring specific files or invoking custom commands
+
+- Documentation:
+  - New [README section](https://github.com/per2jensen/dar-backup?tab=readme-ov-file#limitations-on-file-names-with-special-characters) explains filename restrictions and safe workarounds (e.g. restoring directly with dar, if needed)
+  - Includes a Markdown table listing all disallowed characters
+
+- Test suite:
+  - Existing test cases updated to comply with the new sanitization rules
+  - Additional tests ensure CommandRunner handles large binary output and edge cases cleanly
+
+## v2-beta-0.8.1 - 2025-07-16
+
+Github link: [v2-beta-0.8.1](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.1/v2)
+
+### Added
+
+- FIX: runner now logs an error and fills more data into the returned CommandResult.
+
 ## v2-beta-0.8.0 - 2025-06-13
 
 Github link: [v2-beta-0.8.0](https://github.com/per2jensen/dar-backup/tree/v2-beta-0.8.0/v2)

{dar_backup-0.8.0 → dar_backup-0.8.2}/src/dar_backup/README.md

@@ -9,7 +9,7 @@
 [![PyPI version](https://img.shields.io/pypi/v/dar-backup.svg)](https://pypi.org/project/dar-backup/)
 [![PyPI downloads](https://img.shields.io/badge/dynamic/json?color=blue&label=PyPI%20downloads&query=total&url=https%3A%2F%2Fraw.githubusercontent.com%2Fper2jensen%2Fdar-backup%2Fmain%2Fdownloads.json)](https://pypi.org/project/dar-backup/)
 [![# clones](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/badge_clones.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
-[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)
+[![Milestone](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/per2jensen/dar-backup/main/v2/doc/badges/milestone_badge.json)](https://github.com/per2jensen/dar-backup/blob/main/v2/doc/weekly_clones.png)  <sub>🎯 Stats powered by [ClonePulse](https://github.com/per2jensen/clonepulse)</sub>
 
 The wonderful 'dar' [Disk Archiver](https://github.com/Edrusb/DAR) is used for
 the heavy lifting, together with the [parchive](https://github.com/Parchive/par2cmdline) suite in these scripts.

{dar_backup-0.8.0 → dar_backup-0.8.2}/src/dar_backup/__about__.py

@@ -1,4 +1,4 @@
-__version__ = "0.8.0"
+__version__ = "0.8.2"
 
 __license__ = '''Licensed under GNU GENERAL PUBLIC LICENSE v3, see the supplied file "LICENSE" for details.
 THERE IS NO WARRANTY FOR THE PROGRAM, TO THE EXTENT PERMITTED BY APPLICABLE LAW, not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

dar_backup-0.8.2/src/dar_backup/command_runner.py

@@ -0,0 +1,246 @@
+# SPDX-License-Identifier: GPL-3.0-or-later
+
+import subprocess
+import logging
+import traceback
+import threading
+import os
+import re
+import shlex
+import sys
+import tempfile
+sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), "../src")))
+from typing import List, Optional, Union
+from dar_backup.util import get_logger
+
+
+def is_safe_arg(arg: str) -> bool:
+    """
+    Check if the argument is safe by rejecting dangerous shell characters.
+    """
+    return not re.search(r'[;&|><`$\n]', arg)
+
+
+def sanitize_cmd(cmd: List[str]) -> List[str]:
+    """
+    Validate and sanitize a list of command-line arguments.
+    Ensures all elements are strings and do not contain dangerous shell characters.
+    Raises ValueError if any argument is unsafe.
+    """
+
+    if not isinstance(cmd, list):
+        raise ValueError("Command must be a list of strings")
+    for arg in cmd:
+        if not isinstance(arg, str):
+            raise ValueError(f"Invalid argument type: {arg} (must be string)")
+        if not is_safe_arg(arg):
+            raise ValueError(f"Unsafe argument detected: {arg}")
+    return cmd
+
+def _safe_str(s):
+    if isinstance(s, bytes):
+        return f"<{len(s)} bytes of binary data>"
+    return s
+
+
+class CommandResult:
+    def __init__(
+        self,
+        returncode: int,
+        stdout: Union[str, bytes],
+        stderr: Union[str, bytes],
+        stack: Optional[str] = None,
+        note: Optional[str] = None
+    ):
+        self.returncode = returncode
+        self.stdout = stdout
+        self.stderr = stderr
+        self.stack = stack
+        self.note = note
+
+
+
+    def __repr__(self):
+        return f"<CommandResult returncode={self.returncode}\nstdout={self.stdout}\nstderr={self.stderr}\nstack={self.stack}>"
+
+
+    def __str__(self):
+        return (
+            "CommandResult:\n"
+            f"  Return code: {self.returncode}\n"
+            f"  Note: {self.note if self.note else '<none>'}\n"
+            f"  STDOUT: {_safe_str(self.stdout)}\n"
+            f"  STDERR: {_safe_str(self.stderr)}\n"
+            f"  Stacktrace: {self.stack if self.stack else '<none>'}"
+        )
+
+
+class CommandRunner:
+    def __init__(
+        self,
+        logger: Optional[logging.Logger] = None,
+        command_logger: Optional[logging.Logger] = None,
+        default_timeout: int = 30
+    ):
+        self.logger = logger or get_logger()
+        self.command_logger = command_logger or get_logger(command_output_logger=True)
+        self.default_timeout = default_timeout
+
+        if not self.logger or not self.command_logger:
+            self.logger_fallback()
+
+
+    def logger_fallback(self):
+        """
+        Setup temporary log files
+        """
+        main_log = tempfile.NamedTemporaryFile(delete=False)
+        command_log = tempfile.NamedTemporaryFile(delete=False)
+
+        logger = logging.getLogger("command_runner_fallback_main_logger")
+        command_logger = logging.getLogger("command_runner_fallback_command_logger")
+        logger.setLevel(logging.DEBUG)
+        command_logger.setLevel(logging.DEBUG)
+
+        main_handler = logging.FileHandler(main_log.name)
+        command_handler = logging.FileHandler(command_log.name)
+
+        logger.addHandler(main_handler)
+        command_logger.addHandler(command_handler)
+
+        self.logger = logger
+        self.command_logger = command_logger
+        self.default_timeout = 30
+        self.logger.info("CommandRunner initialized with fallback loggers")
+        self.command_logger.info("CommandRunner initialized with fallback loggers")
+
+        print(f"[WARN] Using fallback loggers:\n  Main log: {main_log.name}\n  Command log: {command_log.name}", file=sys.stderr)
+
+
+
+    def run(
+        self,
+        cmd: List[str],
+        *,
+        timeout: Optional[int] = None,
+        check: bool = False,
+        capture_output: bool = True,
+        text: bool = True
+    ) -> CommandResult:
+        self._text_mode = text 
+        timeout = timeout or self.default_timeout
+
+        cmd_sanitized = None
+
+        try:
+            cmd_sanitized = sanitize_cmd(cmd)
+        except ValueError as e:
+            stack = traceback.format_exc()
+            self.logger.error(f"Command sanitation failed: {e}")
+            return CommandResult(
+                returncode=-1,
+                note=f"Sanitizing failed: command: {' '.join(cmd)}",
+                stdout='',
+                stderr=str(e),
+                stack=stack,
+
+            )
+        finally:
+            cmd = cmd_sanitized
+
+        #command = f"Executing command: {' '.join(cmd)} (timeout={timeout}s)"
+        command = f"Executing command: {' '.join(shlex.quote(arg) for arg in cmd)} (timeout={timeout}s)"
+
+
+        self.command_logger.info(command)
+        self.logger.debug(command)
+
+        stdout_lines = []
+        stderr_lines = []
+
+        try:
+            process = subprocess.Popen(
+                cmd,
+                stdout=subprocess.PIPE if capture_output else None,
+                stderr=subprocess.PIPE if capture_output else None,
+                text=False,
+                bufsize=-1
+            )
+        except Exception as e:
+            stack = traceback.format_exc()
+            return CommandResult(
+                returncode=-1,
+                stdout='',
+                stderr=str(e),
+                stack=stack
+            )
+
+        def stream_output(stream, lines, level):
+            try:
+                while True:
+                    chunk = stream.read(1024)
+                    if not chunk:
+                        break
+                    if self._text_mode:
+                        decoded = chunk.decode('utf-8', errors='replace')
+                        lines.append(decoded)
+                        self.command_logger.log(level, decoded.strip())
+                    else:
+                        lines.append(chunk)
+                        # Avoid logging raw binary data to prevent garbled logs
+            except Exception as e:
+                self.logger.warning(f"stream_output decode error: {e}")
+            finally:
+                stream.close()
+
+
+
+        threads = []
+        if capture_output and process.stdout:
+            t_out = threading.Thread(target=stream_output, args=(process.stdout, stdout_lines, logging.INFO))
+            t_out.start()
+            threads.append(t_out)
+        if capture_output and process.stderr:
+            t_err = threading.Thread(target=stream_output, args=(process.stderr, stderr_lines, logging.ERROR))
+            t_err.start()
+            threads.append(t_err)
+
+        try:
+            process.wait(timeout=timeout)
+        except subprocess.TimeoutExpired:
+            process.kill()
+            self.logger.error(f"Command timed out: {' '.join(cmd)}")
+            return CommandResult(-1, ''.join(stdout_lines), ''.join(stderr_lines))
+        except Exception as e:
+            stack = traceback.format_exc()
+            self.logger.error(f"Command execution failed: {' '.join(cmd)} with error: {e}")
+            return CommandResult(-1, ''.join(stdout_lines), ''.join(stderr_lines), stack)  
+
+        for t in threads:
+            t.join()
+
+
+
+        if self._text_mode:
+            stdout_combined = ''.join(stdout_lines)
+            stderr_combined = ''.join(stderr_lines)
+        else:
+            stdout_combined = b''.join(stdout_lines)
+            stderr_combined = b''.join(stderr_lines)
+
+
+        if check and process.returncode != 0:
+            self.logger.error(f"Command failed with exit code {process.returncode}")
+            return CommandResult(
+                process.returncode,
+                stdout_combined,
+                stderr_combined,
+                stack=traceback.format_stack()
+            )
+
+        return CommandResult(
+            process.returncode,
+                stdout_combined,
+                stderr_combined
+        )
+

{dar_backup-0.8.0 → dar_backup-0.8.2}/src/dar_backup/dar_backup.py

@@ -248,7 +248,7 @@ def verify(args: argparse.Namespace, backup_file: str, backup_definition: str, c
         PermissionError: If a permission error occurs while comparing files.
     """
     result = True
-    command = ['dar', '-t', backup_file, '-Q']
+    command = ['dar', '-t', backup_file, '-N', '-Q']
  
  
     log_basename = os.path. dirname(config_settings.logfile_location)
@@ -315,7 +315,7 @@ def verify(args: argparse.Namespace, backup_file: str, backup_definition: str, c
     for restored_file_path in random_files:
         try:
             args.verbose and logger.info(f"Restoring file: '{restored_file_path}' from backup to: '{config_settings.test_restore_dir}' for file comparing")
-            command = ['dar', '-x', backup_file, '-g', restored_file_path.lstrip("/"), '-R', config_settings.test_restore_dir, '-Q', '-B', args.darrc, 'restore-options']
+            command = ['dar', '-x', backup_file, '-g', restored_file_path.lstrip("/"), '-R', config_settings.test_restore_dir, '--noconf',  '-Q', '-B', args.darrc, 'restore-options']
             args.verbose and logger.info(f"Running command: {' '.join(map(shlex.quote, command))}")
             process = runner.run(command, timeout = config_settings.command_timeout_secs)    
             if process.returncode != 0:
@@ -347,7 +347,7 @@ def restore_backup(backup_name: str, config_settings: ConfigSettings, restore_di
     results: List[tuple] = []
     try:
         backup_file = os.path.join(config_settings.backup_dir, backup_name)
-        command = ['dar', '-x', backup_file, '-Q', '-D']
+        command = ['dar', '-x', backup_file, '--noconf', '-Q', '-D']
         if restore_dir:
             if not os.path.exists(restore_dir):
                 os.makedirs(restore_dir)
@@ -390,7 +390,7 @@ def get_backed_up_files(backup_name: str, backup_dir: str):
     logger.debug(f"Getting backed up files in xml from DAR archive: '{backup_name}'")
     backup_path = os.path.join(backup_dir, backup_name)
     try:
-        command = ['dar', '-l', backup_path, '-am', '-as', "-Txml" , '-Q']
+        command = ['dar', '-l', backup_path, '--noconf', '-am', '-as', "-Txml" , '-Q']
         logger.debug(f"Running command: {' '.join(map(shlex.quote, command))}")
         command_result = runner.run(command)
         # Parse the XML data
@@ -418,7 +418,7 @@ def list_contents(backup_name, backup_dir, selection=None):
     backup_path = os.path.join(backup_dir, backup_name)
 
     try:
-        command = ['dar', '-l', backup_path, '-am', '-as', '-Q']
+        command = ['dar', '-l', backup_path, '--noconf',  '-am', '-as', '-Q']
         if selection:
             selection_criteria = shlex.split(selection)
             command.extend(selection_criteria)

dar_backup-0.8.0/src/dar_backup/command_runner.py

@@ -1,133 +0,0 @@
-# SPDX-License-Identifier: GPL-3.0-or-later
-
-import subprocess
-import logging
-import threading
-import os
-import sys
-import tempfile
-sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), "../src")))
-from typing import List, Optional
-from dar_backup.util import get_logger
-
-
-class CommandResult:
-    def __init__(self, returncode: int, stdout: str, stderr: str):
-        self.returncode = returncode
-        self.stdout = stdout
-        self.stderr = stderr
-
-    def __repr__(self):
-        return f"<CommandResult returncode={self.returncode}>"
-
-
-class CommandRunner:
-    def __init__(
-        self,
-        logger: Optional[logging.Logger] = None,
-        command_logger: Optional[logging.Logger] = None,
-        default_timeout: int = 30
-    ):
-        self.logger = logger or get_logger()
-        self.command_logger = command_logger or get_logger(command_output_logger=True)
-        self.default_timeout = default_timeout
-
-        if not self.logger or not self.command_logger:
-            self.logger_fallback()
-
-    def logger_fallback(self):
-        """
-        Setup temporary log files
-        """
-        main_log = tempfile.NamedTemporaryFile(delete=False)
-        command_log = tempfile.NamedTemporaryFile(delete=False)
-
-        logger = logging.getLogger("command_runner_fallback_main_logger")
-        command_logger = logging.getLogger("command_runner_fallback_command_logger")
-        logger.setLevel(logging.DEBUG)
-        command_logger.setLevel(logging.DEBUG)
-
-        main_handler = logging.FileHandler(main_log.name)
-        command_handler = logging.FileHandler(command_log.name)
-
-        logger.addHandler(main_handler)
-        command_logger.addHandler(command_handler)
-
-        self.logger = logger
-        self.command_logger = command_logger
-        self.default_timeout = 30
-        self.logger.info("CommandRunner initialized with fallback loggers")
-        self.command_logger.info("CommandRunner initialized with fallback loggers")
-
-        print(f"[WARN] Using fallback loggers:\n  Main log: {main_log.name}\n  Command log: {command_log.name}", file=sys.stderr)
-
-
-
-    def run(
-        self,
-        cmd: List[str],
-        *,
-        timeout: Optional[int] = None,
-        check: bool = False,
-        capture_output: bool = True,
-        text: bool = True
-    ) -> CommandResult:
-        timeout = timeout or self.default_timeout
-
-        #log the command to be executed
-        command = f"Executing command: {' '.join(cmd)} (timeout={timeout}s)"
-        self.command_logger.info(command) # log to command logger
-        self.logger.debug(command)        # log to main logger if "--log-level debug"
-
-        process = subprocess.Popen(
-            cmd,
-            stdout=subprocess.PIPE if capture_output else None,
-            stderr=subprocess.PIPE if capture_output else None,
-            text=False,
-            bufsize=-1
-        )
-
-        stdout_lines = []
-        stderr_lines = []
-
-
-        def stream_output(stream, lines, level):
-            try:
-                while True:
-                    chunk = stream.read(1024)
-                    if not chunk:
-                        break
-                    decoded = chunk.decode('utf-8', errors='replace')
-                    lines.append(decoded)
-                    self.command_logger.log(level, decoded.strip())
-            except Exception as e:
-                self.logger.warning(f"stream_output decode error: {e}")
-            finally:
-                stream.close()
-
-
-
-        threads = []
-        if capture_output and process.stdout:
-            t_out = threading.Thread(target=stream_output, args=(process.stdout, stdout_lines, logging.INFO))
-            t_out.start()
-            threads.append(t_out)
-        if capture_output and process.stderr:
-            t_err = threading.Thread(target=stream_output, args=(process.stderr, stderr_lines, logging.ERROR))
-            t_err.start()
-            threads.append(t_err)
-
-        try:
-            process.wait(timeout=timeout)
-        except subprocess.TimeoutExpired:
-            process.kill()
-            self.logger.error(f"Command timed out: {' '.join(cmd)}")
-            return CommandResult(-1, ''.join(stdout_lines), ''.join(stderr_lines))
-
-        for t in threads:
-            t.join()
-
-        if check and process.returncode != 0:
-            self.logger.error(f"Command failed with exit code {process.returncode}")
-
-        return CommandResult(process.returncode, ''.join(stdout_lines), ''.join(stderr_lines))