Path permission fix

Author: davidhozic

docs/source/changelog.rst

@@ -45,6 +45,10 @@ v2.7
   - Invite link tracking
   - :class:`~daf.guild.GUILD`: ``invite_track`` parameter for tracking invite links
 
+- File outputs:
+
+  - Changed all paths' defaults to be stored under /<user-home-dir>/daf/ folder to prevent permission problems
+
 - :class:`~daf.guild.AutoGUILD` ``interval`` default changed to ``timedelta(minutes=1)``
 - xMESSAGE ``start_in`` now accepts :class:`datetime.datetime` - send at specific datetime.
 - GUI:

src/daf/core.py

@@ -37,7 +37,8 @@
 CORE_TASK_SLEEP_SEC = 0.1
 ACCOUNT_CLEANUP_DELAY = 10
 SCHEMA_BACKUP_DELAY = 120
-SHILL_LIST_BACKUP_PATH = Path("objects.sbf")  # sbf -> Schema Backup File
+DAF_PATH = Path.home().joinpath("daf")
+SHILL_LIST_BACKUP_PATH = DAF_PATH.joinpath("objects.sbf")  # sbf -> Schema Backup File
 # ---------------------------------------
 
 
@@ -79,6 +80,7 @@ async def schema_backup_task():
         loop.call_later(SCHEMA_BACKUP_DELAY, event.set)
         await event.wait()
         event.clear()
+        DAF_PATH.mkdir(parents=True, exist_ok=True)
         tmp_path = str(SHILL_LIST_BACKUP_PATH) + ".1"
         trace("Saving objects to file.", TraceLEVELS.DEBUG)
         try:
@@ -118,7 +120,7 @@ async def schema_load_from_file() -> None:
 async def initialize(user_callback: Optional[Union[Callable, Coroutine]] = None,
                      debug: Optional[Union[TraceLEVELS, int, str]] = TraceLEVELS.NORMAL,
                      logger: Optional[logging.LoggerBASE] = None,
-                     accounts: List[client.ACCOUNT] = [],
+                     accounts: List[client.ACCOUNT] = None,
                      save_to_file: bool = False) -> None:
     """
     The main initialization function.
@@ -132,6 +134,8 @@ async def initialize(user_callback: Optional[Union[Callable, Coroutine]] = None,
         Parameters are the same as in :func:`daf.core.run`.
     """
     loop = asyncio.get_event_loop()
+    if accounts is None:
+        accounts = []
     # ------------------------------------------------------------
     # Initialize tracing
     # ------------------------------------------------------------
@@ -143,7 +147,7 @@ async def initialize(user_callback: Optional[Union[Callable, Coroutine]] = None,
     # Initialize logging
     # ------------------------------------------------------------
     if logger is None:
-        logger = logging.LoggerJSON(path="History")
+        logger = logging.LoggerJSON(path=str(Path.home().joinpath("daf/History")))
 
     await logging.initialize(logger)
     # ------------------------------------------------------------
@@ -401,7 +405,7 @@ def _shutdown_clean(loop: asyncio.AbstractEventLoop) -> None:
 def run(user_callback: Optional[Union[Callable, Coroutine]] = None,
         debug: Optional[Union[TraceLEVELS, int, str, bool]] = TraceLEVELS.NORMAL,
         logger: Optional[logging.LoggerBASE] = None,
-        accounts: Optional[List[client.ACCOUNT]] = [],
+        accounts: Optional[List[client.ACCOUNT]] = None,
         save_to_file: bool = False) -> None:
     """
     .. versionchanged:: 2.7
@@ -428,7 +432,7 @@ def run(user_callback: Optional[Union[Callable, Coroutine]] = None,
         The higher value this option is, the more will be displayed.
     logger: Optional[loggers.LoggerBASE]
         The logging class to use.
-        If this is not provided, JSON is automatically used.
+        If this is not provided, JSON is automatically used with the ``path`` parameter set to /<user-home-dir>/daf/History
     accounts: Optional[List[client.ACCOUNT]]
         List of :class:`~daf.client.ACCOUNT` (Discord accounts) to use.
         .. versionadded:: v2.4

src/daf/logging/_logging.py

@@ -292,7 +292,7 @@ async def _save_log(
                         .joinpath("{:02d}".format(timestruct.month))
                         .joinpath("{:02d}".format(timestruct.day)))
 
-        logging_output.mkdir(parents=True,exist_ok=True)
+        logging_output.mkdir(parents=True, exist_ok=True)
         logging_output = logging_output.joinpath("".join(char if char not in C_FILE_NAME_FORBIDDEN_CHAR
                                                               else "#" for char in guild_context["name"]) + ".json")          
         # Create file if it doesn't exist

src/daf/logging/sql/mgr.py

@@ -7,6 +7,7 @@
 from datetime import datetime, date
 from typing import Callable, Dict, List, Literal, Any, Union, Optional, Tuple
 from contextlib import suppress
+from pathlib import Path
 from typeguard import typechecked
 
 from ..tracing import TraceLEVELS, trace
@@ -209,7 +210,9 @@ def exists(self, key: Any) -> bool:
 class LoggerSQL(logging.LoggerBASE):
     """
     .. versionchanged:: v2.7
-        Invite link tracking.
+
+        - Invite link tracking.
+        - Default database file output set to /<user-home-dir>/daf/messages
 
     Used for controlling the SQL database used for message logs.
 
@@ -270,11 +273,12 @@ def __init__(self,
         self.password = password
         self.server = server
         self.port = port
-        self.database = database if database is not None else "messages"
+        self.database = database if database is not None else str(Path.home().joinpath("daf/messages"))
         self.dialect = dialect
 
         if self.dialect == "sqlite":
             self.database += ".db"
+            Path(self.database).parent.mkdir(parents=True, exist_ok=True)
 
         # Set in ._begin_engine
         self.engine: sqa.engine.Engine = None
@@ -706,6 +710,9 @@ async def _handle_error(self,
                 res = False
                 await self._reconnect_after(SQL_RECONNECT_TIME)
             else:
+                if self.dialect == "sqlite":
+                    Path(self.database).parent.mkdir(parents=True, exist_ok=True)
+
                 await self._create_tables()
                 self._clear_caches()
                 await self._generate_lookup_values()

src/daf/web.py

@@ -65,7 +65,7 @@ class GLOBALS:
 WD_TIMEOUT_LONG = 90
 WD_RD_CLICK_UPPER_N = 5
 WD_RD_CLICK_LOWER_N = 2
-WD_OUTPUT_PATH = pathlib.Path("./daf_web_data")
+WD_OUTPUT_PATH = pathlib.Path.home().joinpath("daf/daf_web_data")
 WD_TOKEN_PATH = WD_OUTPUT_PATH.joinpath("tokens.json")
 WD_PROFILES_PATH = WD_OUTPUT_PATH.joinpath("chrome_profiles")
 WD_QUERY_SLEEP_S = 5
@@ -436,7 +436,7 @@ async def initialize(self) -> None:
         Any
             Raised in :py:meth:`~SeleniumCLIENT.login` method.
         """
-        WD_OUTPUT_PATH.mkdir(exist_ok=True)
+        WD_OUTPUT_PATH.mkdir(exist_ok=True, parents=True)
         web_data_path = pathlib.Path(WD_PROFILES_PATH, self._username)
 
         opts = Options()

src/daf_gui/main.py

@@ -5,6 +5,7 @@
 import sys
 
 from importlib.util import find_spec
+from pathlib import Path
 
 installed = find_spec("ttkbootstrap") is not None
 
@@ -258,9 +259,9 @@ def import_accounts():
         self.combo_tracing.pack(fill=tk.X, side="left", expand=True)
 
         self.combo_logging_mgr["values"] = [
-            ObjectInfo(daf.LoggerJSON, {"path": "History"}),
-            ObjectInfo(daf.LoggerSQL, {}),
-            ObjectInfo(daf.LoggerCSV, {"path": "History", "delimiter": ";"}),
+            ObjectInfo(daf.LoggerJSON, {"path": str(Path.home().joinpath("daf/History"))}),
+            ObjectInfo(daf.LoggerSQL, {"database": str(Path.home().joinpath("daf/messages")), "dialect": "sqlite"}),
+            ObjectInfo(daf.LoggerCSV, {"path": str(Path.home().joinpath("daf/History")), "delimiter": ";"}),
         ]
 
         self.combo_tracing["values"] = [en for en in daf.TraceLEVELS]