Issue #198 Only install and use oschmod when it is a Windows system.

Author: JohanKJSchreurs

openeo/rest/auth/config.py

@@ -14,12 +14,14 @@
 from pathlib import Path
 from typing import Union, Tuple, Dict
 
-import oschmod
-
 from openeo import __version__
 from openeo.config import get_user_config_dir, get_user_data_dir
 from openeo.util import rfc3339, deep_get, deep_set
 
+if platform.system() == 'Windows':
+    import oschmod
+
+
 _PRIVATE_PERMS = stat.S_IRUSR | stat.S_IWUSR
 
 log = logging.getLogger(__name__)

setup.py

@@ -47,7 +47,7 @@
           'xarray>=0.12.3',
           'pandas>0.20.0',
           'deprecated>=1.2.12',
-          'oschmod',
+          'oschmod; sys_platform == "win32"',
           'pywin32; sys_platform == "win32"',
       ],
       extras_require={

tests/rest/auth/test_config.py

@@ -2,12 +2,18 @@
 from unittest import mock
 import platform
 
-import oschmod
 import pytest
 
 import openeo.rest.auth.config
 from openeo.rest.auth.config import RefreshTokenStore, AuthConfig, PrivateJsonFile
 
+# TODO: We could simplify the code if we can verify oschmod works transparently on Linux/Mac as well.
+#   I think oschmod does work on Linux and Mac, but since it we use it for a
+#   security sensitive feature, let's make sure there are no problems first.
+#   For now, it is fine to have some Windows specific code.
+if platform.system() == 'Windows':
+    import oschmod
+
 
 class TestPrivateJsonFile: