Wheel Speed Controller Permission Denied Error (#494)

* Bug fixed: wrong format of permission. (#493)

* Added error catches for LTM. (#493)

* Added pre-check for write permission. (#493)

* Added debug info. (#493)

* Bug fixed. (#493)

* Bug fixed. (#493)

* Added debug info. (#493)

* Hinted the permission issue. (#493)

* Merged submodules. (#493)

* Merged submodules. (#493)

Author: ATATC

leads/ltm.py

@@ -1,23 +1,54 @@
 from json import loads as _loads, dumps as _dumps
-from os.path import abspath as _abspath
+from os import chmod as _chmod, access as _access, R_OK as _R_OK, W_OK as _W_OK
+from os.path import abspath as _abspath, exists as _exists
 
+from leads.logger import L
 from leads.types import SupportedConfigValue as _SupportedConfigValue
 
+_PATH: str = f"{_abspath(__file__)[:-6]}_ltm/core"
+
 _ltm: dict[str, _SupportedConfigValue] = {}
 
 
+def _acquire_permission() -> bool:
+    if _access(_PATH, _R_OK) and _access(_PATH, _W_OK):
+        return True
+    L.debug(f"Attempting to acquire permission for {_PATH}")
+    try:
+        _chmod(_PATH, 0o666)
+        return True
+    except Exception as e:
+        L.debug(f"Failed to acquire permission: {repr(e)}")
+        L.debug(f"For Linux users, try executing `sudo chmod 666 {_PATH}` manually or run LEADS as the root user")
+        return False
+
+
 def _load_ltm() -> None:
+    if not _permission_ok:
+        return
     global _ltm
-    with open(f"{_abspath(__file__)[:-6]}_ltm/core") as f:
-        ltm_content = f.read()
-        if not (ltm_content.startswith("{") and ltm_content.endswith("}")):
-            ltm_content = "{}"
-        _ltm = _loads(ltm_content)
+    try:
+        if not _exists(_PATH):
+            with open(_PATH, "w") as f:
+                f.write("{}")
+            return
+        with open(_PATH) as f:
+            ltm_content = f.read()
+            if not (ltm_content.startswith("{") and ltm_content.endswith("}")):
+                ltm_content = "{}"
+            _ltm = _loads(ltm_content)
+    except Exception as e:
+        L.warn(f"Attempted but failed to load LTM: {repr(e)}")
 
 
 def _sync_ltm() -> None:
-    with open(f"{_abspath(__file__)[:-6]}_ltm/core", "w") as f:
-        f.write(_dumps(_ltm))
+    if not _permission_ok:
+        return
+    try:
+        with open(_PATH, "w") as f:
+            f.write(_dumps(_ltm))
+    except Exception as e:
+        L.warn(f"Attempted but failed to sync LTM: {repr(e)}")
 
 
 def ltm_get(key: str) -> _SupportedConfigValue:
@@ -29,4 +60,6 @@ def ltm_set(key: str, value: _SupportedConfigValue) -> None:
     _sync_ltm()
 
 
+_permission_ok: bool = _acquire_permission()
+L.debug(f"LTM permission {"OK" if _permission_ok else "NOT OK"}: {_PATH}")
 _load_ltm()

leads_vec/_bootloader/systemd.py

@@ -16,7 +16,7 @@ def register_leads_vec() -> None:
             _mkdirs("/usr/local/leads")
         with open("/usr/local/leads/config.json", "w") as f:
             f.write(str(Config({})))
-    _chmod("/usr/local/leads/config.json", 0x644)
+    _chmod("/usr/local/leads/config.json", 0o644)
     _chmod(script := f"{_abspath(__file__)[:-10]}leads-vec.service.sh", 0o755)
     if not _exists(user_systemd := f"/home/{(username := _get_login())}/.config/systemd/user"):
         _L.debug(f"User Systemd not found. Creating \"{user_systemd}\"...")

readthedocs

@@ -32,5 +32,6 @@ execute_root "chmod" "+x" "/bin/leads-vec-rc"
 execute_root "echo" "#!/bin/bash" > "/bin/leads-vec-dp"
 execute_root "echo" 'python-leads -m leads_vec_dp "$@"' >> "/bin/leads-vec-dp"
 execute_root "chmod" "+x" "/bin/leads-vec-dp"
+execute_root "chmod" "666" "/usr/local/leads/venv/lib/python3.12/site-packages/leads/_ltm/core"
 echo "Verifying..."
 execute "leads-vec" "info"