Repository navigation
Permissions issues with PrivateJsonFile on windows #198
Description
Windows does not support fine grained linux style permission chmoding, causing perrmission issues like
File "D:\tmp\env\lib\site-packages\openeo\rest\auth\config.py", line 85, in _write
assert_private_file(self._path)
File "D:\tmp\env\lib\site-packages\openeo\rest\auth\config.py", line 30, in assert_private_file
p=path, a=mode, e=_PRIVATE_PERMS)
PermissionError: File C:\Users\john\AppData\Local\Temp\win-perms-test.json is readable by others: st_mode 100666 (expected permissions: 600).```
Activity
Issue #198: only warn about file permissions on windows
soxofaan commentedon Mar 29, 2021
did a first quick fix to make OIDC at least usable on Windows:
now a warning is raised when permissions are not correctly set
soxofaan commentedon Mar 29, 2021
next step: investigate oschmod (https://pypi.org/project/oschmod/) to set permissions on windows fully (which does not work with vanilla
os.chmod)Issue #198/#249 `log.info` instead of warn about config file permissi…
soxofaan commentedon Oct 11, 2021
Also interesting solution: some kind of logout feature to remove these local files: #250
soxofaan commentedon Dec 13, 2022
FYI failing tests related to this are blocker for
Issue #198 Use oschmod on Windows to set/get permissions of PrivateJs…
Issue #198 Fix bug introduced in previous commit: tests/rest/auth/tes…
Issue #198 Only install and use oschmod when it is a Windows system.
Issue #198 Code review: simplify and encapsulate getting/setting file…
Issue #198 Correct regex in test_config.py::test_get_config_verbose
Issue #198 Code review: make dependency oschmod required on Windows b…
Issue #198: Updated changelog
Issue #198 Use oschmod on Windows to set/get permissions of PrivateJs…
Issue #198 Fix bug introduced in previous commit: tests/rest/auth/tes…
Issue #198 Only install and use oschmod when it is a Windows system.
Issue #198 Code review: simplify and encapsulate getting/setting file…
Issue #198 Correct regex in test_config.py::test_get_config_verbose
Issue #198 Code review: make dependency oschmod required on Windows b…
Issue #198: Updated changelog
Issue #198 minor finetuning
soxofaan commentedon Jan 4, 2023
PR #351 was merged