Possibility for no authentication (#464)

anikaweinmann · mmacata · web-flow · commit 5c83d834e09c · 2023-08-31T12:13:46.000+02:00
* add a no-auth user
* tests running with right config
* Apply suggestions from code review

Co-authored-by: Carmen Tawalika &lt;mmacata@users.noreply.github.com&gt;
diff --git a/.dockerignore b/.dockerignore
@@ -5,6 +5,7 @@ docker
 !docker/actinia-core-dev/actinia.cfg
 !docker/actinia-core-dev/endpoints.csv
 !docker/actinia-core-tests/actinia-test.cfg
+!docker/actinia-core-tests/actinia-test-noauth.cfg
 .github
 .travis
 .travis.yml
diff --git a/.flake8 b/.flake8
@@ -3,6 +3,7 @@
 # E402 module level import not at top of file
 # E501 line too long (80 > 79 characters)
 # F401 '.health_check.health_check' imported but unused
+# F811 redefinition of unused 'verify_password' from line 55
 # F821 undefined name 'QFile'
 # W605 invalid escape sequence '\<'
 
@@ -30,6 +31,7 @@ per-file-ignores =
     ./src/actinia_core/core/interim_results.py: W605
     ./src/actinia_core/core/list_grass_modules.py: F821
     ./src/actinia_core/testsuite.py: F401
+    ./src/actinia_core/rest/base/user_auth.py: F811
     ./src/actinia_core/rest/ephemeral_processing.py: W605
     ./src/actinia_core/processing/actinia_processing/ephemeral_renderer_base/*: E501
     ./src/actinia_core/processing/actinia_processing/ephemeral_with_export/raster_export.py: E501
diff --git a/.gitignore b/.gitignore
@@ -52,3 +52,4 @@ docker/redis_queue_data/dump.rdb
 !docker/actinia-core-dev/actinia.cfg
 !docker/actinia-core-dev/endpoints.csv
 !docker/actinia-core-tests/actinia-test.cfg
+!docker/actinia-core-tests/actinia-test-noauth.cfg
diff --git a/Makefile b/Makefile
@@ -18,5 +18,8 @@ unittest:
 devtest:
 	sh ./tests_with_redis.sh dev
 
+noauthtest:
+		sh ./tests_with_redis.sh noauth
+
 integrationtest:
 	sh ./tests_with_redis.sh integrationtest
diff --git a/docker/actinia-core-tests/Dockerfile b/docker/actinia-core-tests/Dockerfile
@@ -40,6 +40,7 @@ RUN grass --tmp-location EPSG:4326 --exec g.extension -s \
 # copy needed files and configs for test
 COPY docker/actinia-core-alpine/actinia.cfg /etc/default/actinia
 COPY docker/actinia-core-tests/actinia-test.cfg /etc/default/actinia_test
+COPY docker/actinia-core-tests/actinia-test-noauth.cfg /etc/default/actinia_test_noauth
 
 RUN pip3 install pytest pytest-cov
 
diff --git a/docker/actinia-core-tests/actinia-test-noauth.cfg b/docker/actinia-core-tests/actinia-test-noauth.cfg
@@ -0,0 +1,26 @@
+[GRASS]
+grass_database = /actinia_core/grassdb
+grass_user_database = /actinia_core/userdata
+grass_tmp_database = /actinia_core/workspace/temp_db
+grass_resource_dir = /actinia_core/resources
+
+[API]
+force_https_urls = True
+authentication = False
+
+[LIMITS]
+max_cell_limit = 22500000
+process_time_limt = 60
+process_num_limit = 20
+
+[REDIS]
+worker_logfile = /actinia_core/workspace/tmp/actinia_worker_test.log
+redis_server_url = localhost
+redis_server_port = 6379
+
+[LOGGING]
+log_level = 1
+
+[MISC]
+tmp_workdir = /actinia_core/workspace/tmp
+download_cache = /actinia_core/workspace/download_cache
diff --git a/docker/actinia-core-tests/actinia-test.cfg b/docker/actinia-core-tests/actinia-test.cfg
@@ -1,43 +1,25 @@
 [GRASS]
-grass_gis_start_script = /usr/local/bin/grass
 grass_database = /actinia_core/grassdb
 grass_user_database = /actinia_core/userdata
 grass_tmp_database = /actinia_core/workspace/temp_db
 grass_resource_dir = /actinia_core/resources
-grass_addon_path = /root/.grass8/addons/
-grass_gis_base = /usr/local/grass
-grass_modules_xml_path = /usr/local/grass/gui/wxpython/xml/module_items.xml
-grass_default_location = nc_spm_08
 
 [API]
-plugins = []
 force_https_urls = True
 
 [LIMITS]
 max_cell_limit = 22500000
 process_time_limt = 60
 process_num_limit = 20
-number_of_workers = 3
 
 [REDIS]
+worker_logfile = /actinia_core/workspace/tmp/actinia_worker_test.log
 redis_server_url = localhost
 redis_server_port = 6379
-redis_queue_server_url = localhost
-redis_queue_server_port = 6379
-worker_queue_name = actinia_job
-worker_logfile = /actinia_core/workspace/tmp/actinia_worker_test.log
 
 [LOGGING]
-log_interface = fluentd
-log_fluent_host = fluentd
-log_fluent_port = 24224
 log_level = 1
 
 [MISC]
 tmp_workdir = /actinia_core/workspace/tmp
 download_cache = /actinia_core/workspace/download_cache
-secret_key = token_signing_key_changeme
-
-[MANAGEMENT]
-default_user = user
-default_user_group = group
diff --git a/pyproject.toml b/pyproject.toml
@@ -98,11 +98,12 @@ API_Docs = "https://redocly.github.io/redoc/?url=https://actinia.mundialis.de/la
 
 [tool.pytest.ini_options]
 minversion = "6.0"
-addopts = "--cov actinia_core --cov-report term-missing --verbose --tb=line -x"
+addopts = "--cov actinia_core --cov-report term-missing --verbose --tb=line -x -s"
 testpaths = [
     "tests",
 ]
 markers = [
     "dev: test current in development",
     "unittest: completely independent test",
+    "noauth: tests for actinia without authentication",
 ]
diff --git a/src/actinia_core/core/common/app.py b/src/actinia_core/core/common/app.py
@@ -4,7 +4,7 @@
 # performance processing of geographical data that uses GRASS GIS for
 # computational tasks. For details, see https://actinia.mundialis.de/
 #
-# Copyright (c) 2016-2018 Sören Gebbert and mundialis GmbH & Co. KG
+# Copyright (c) 2016-2023 Sören Gebbert and mundialis GmbH & Co. KG
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -116,9 +116,9 @@
 from actinia_api import API_VERSION, URL_PREFIX
 
 __license__ = "GPLv3"
-__author__ = "Sören Gebbert, Julia Haas"
+__author__ = "Sören Gebbert, Julia Haas, Anika Weinmann"
 __copyright__ = (
-    "Copyright 2016-2021, Sören Gebbert and mundialis GmbH & Co. KG"
+    "Copyright 2016-2023, Sören Gebbert and mundialis GmbH & Co. KG"
 )
 __maintainer__ = "mundialis"
 
@@ -165,11 +165,14 @@
             }
         }
     ]
-else:
+elif global_config.AUTHENTICATION:
     # Set the security definition in an unconventional way
     flask_api._swagger_object["securityDefinitions"] = {
         "basicAuth": {"type": "basic"}
     }
     flask_api._swagger_object["security"] = [{"basicAuth": []}]
 
     auth = HTTPBasicAuth()
+else:
+    # No authentication
+    auth = HTTPBasicAuth()
diff --git a/src/actinia_core/core/common/config.py b/src/actinia_core/core/common/config.py
@@ -4,7 +4,7 @@
 # performance processing of geographical data that uses GRASS GIS for
 # computational tasks. For details, see https://actinia.mundialis.de/
 #
-# Copyright (c) 2016-2018 Sören Gebbert and mundialis GmbH & Co. KG
+# Copyright (c) 2016-2023 Sören Gebbert and mundialis GmbH & Co. KG
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -34,7 +34,7 @@
 __license__ = "GPLv3"
 __author__ = "Sören Gebbert, Anika Weinmann"
 __copyright__ = (
-    "Copyright 2016-2022, Sören Gebbert and mundialis GmbH & Co. KG"
+    "Copyright 2016-2023, Sören Gebbert and mundialis GmbH & Co. KG"
 )
 __maintainer__ = "mundialis GmbH & Co. KG"
 
@@ -320,6 +320,8 @@ def __init__(self):
         self.PLUGINS = []
         # ENDPOINTS_CONFIG: configuration csv file for endpoints
         self.ENDPOINTS_CONFIG = None
+        # AUTHENTICATION: If set False no authentication is needed
+        self.AUTHENTICATION = True
 
         """
         KEYCLOAK: has only to be set if keycloak server is configured with
@@ -560,6 +562,7 @@ def write(self, path=DEFAULT_CONFIG_PATH):
         config.set("API", "FORCE_HTTPS_URLS", str(self.FORCE_HTTPS_URLS))
         config.set("API", "PLUGINS", str(self.PLUGINS))
         config.set("API", "ENDPOINTS_CONFIG", str(self.ENDPOINTS_CONFIG))
+        config.set("API", "AUTHENTICATION", str(self.AUTHENTICATION))
 
         config.add_section("KEYCLOAK")
         config.set(
@@ -810,6 +813,10 @@ def read(self, path=DEFAULT_CONFIG_PATH):
                     self.ENDPOINTS_CONFIG = config.get(
                         "API", "ENDPOINTS_CONFIG"
                     )
+                if config.has_option("API", "AUTHENTICATION"):
+                    self.AUTHENTICATION = config.getboolean(
+                        "API", "AUTHENTICATION"
+                    )
 
             if config.has_section("KEYCLOAK"):
                 if config.has_option("KEYCLOAK", "CONFIG_PATH"):
diff --git a/src/actinia_core/core/common/user_noauth.py b/src/actinia_core/core/common/user_noauth.py
@@ -0,0 +1,80 @@
+# -*- coding: utf-8 -*-
+#######
+# actinia-core - an open source REST API for scalable, distributed, high
+# performance processing of geographical data that uses GRASS GIS for
+# computational tasks. For details, see https://actinia.mundialis.de/
+#
+# Copyright (c) 2023 mundialis GmbH & Co. KG
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <https://www.gnu.org/licenses/>.
+#
+#######
+
+"""
+User management for no authentication
+"""
+
+# from passlib.apps import custom_app_context as pwd_context
+# import jwt
+# from datetime import datetime, timezone, timedelta
+from actinia_core.core.common.config import global_config
+from actinia_core.core.redis_user import redis_user_interface
+from actinia_core.core.common.user import ActiniaUser
+
+
+__author__ = "Anika Weinmann"
+__copyright__ = "Copyright 2023, mundialis GmbH & Co. KG"
+__maintainer__ = "mundialis GmbH & Co. KG"
+
+
+class ActiniaUserNoAuth(ActiniaUser):
+    """
+    The Actinia Core user management class for no authentication
+    """
+
+    db = redis_user_interface
+
+    def __init__(self):
+        """Constructor
+        Initialize and create a user object for no authentication.
+        """
+        self.user_id = global_config.DEFAULT_USER
+        self.user_group = global_config.DEFAULT_USER_GROUP
+        self.password_hash = None
+        self.user_role = "superadmin"
+        self.permissions = None
+        self.cell_limit = global_config.MAX_CELL_LIMIT
+        self.accessible_datasets = {
+            "nc_spm_08": ["PERMANENT", "user1", "landsat"],
+            "ECAD": ["PERMANENT"],
+            "latlong_wgs84": ["PERMANENT"],
+        }
+        self.accessible_modules = global_config.MODULE_ALLOW_LIST
+        self.process_num_limit = global_config.PROCESS_NUM_LIMIT
+        self.process_time_limit = global_config.PROCESS_TIME_LIMT
+
+    @staticmethod
+    def create_user():
+        """Create a new user object for no authentication and initialize it
+
+        Returns:
+            actinia_core_api.common.user_noauth.ActiniaUserNoAuth:
+            A new user object in case of success, or None in case of failure
+        """
+        user = ActiniaUserNoAuth()
+        user.hash_password("")
+
+        if user.commit() is True:
+            return user
+        return None
diff --git a/src/actinia_core/rest/base/user_auth.py b/src/actinia_core/rest/base/user_auth.py
@@ -5,7 +5,7 @@
 # performance processing of geographical data that uses GRASS GIS for
 # computational tasks. For details, see https://actinia.mundialis.de/
 #
-# Copyright (c) 2016-2018 Sören Gebbert and mundialis GmbH & Co. KG
+# Copyright (c) 2016-2023 Sören Gebbert and mundialis GmbH & Co. KG
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -33,16 +33,29 @@
 from actinia_core.core.common.app import auth
 from actinia_core.core.common.keycloak_user import ActiniaKeycloakUser
 from actinia_core.core.common.user import ActiniaUser
+from actinia_core.core.common.user_noauth import ActiniaUserNoAuth
 from actinia_core.core.messages_logger import MessageLogger
 
 __license__ = "GPLv3"
-__author__ = "Sören Gebbert, Julia Haas"
+__author__ = "Sören Gebbert, Julia Haas, Anika Weinmann"
 __copyright__ = (
-    "Copyright 2016-2022, Sören Gebbert and mundialis GmbH & Co. KG"
+    "Copyright 2016-2023, Sören Gebbert and mundialis GmbH & Co. KG"
 )
 __maintainer__ = "mundialis"
 
 
+if not global_config.AUTHENTICATION:
+    # No authentication
+    @auth.login_required
+    def login_required(a, b):
+        return False
+
+    @auth.verify_password
+    def verify_password(username_or_token, password):
+        g.user = ActiniaUserNoAuth.create_user()
+        return True
+
+
 if global_config.KEYCLOAK_CONFIG_PATH:
 
     @auth.verify_token
@@ -63,7 +76,7 @@ def verify_token(token):
         return True
 
 
-if global_config.KEYCLOAK_CONFIG_PATH is None:
+if global_config.KEYCLOAK_CONFIG_PATH is None and global_config.AUTHENTICATION:
 
     @auth.verify_password
     def verify_password(username_or_token, password):
diff --git a/tests/test_common_base.py b/tests/test_common_base.py
@@ -103,7 +103,6 @@ class CommonTestCaseBase(unittest.TestCase):
     def setUpClass(cls):
         if custom_actinia_cfg is not False:
             global_config.read(custom_actinia_cfg)
-            print(global_config)
         else:
             global_config.REDIS_SERVER_URL = "localhost"
             global_config.REDIS_SERVER_PORT = 7000
diff --git a/tests/test_noauth.py b/tests/test_noauth.py
diff --git a/tests_with_redis.sh b/tests_with_redis.sh

Original file line number	Diff line number	Diff line change
`@@ -98,11 +98,12 @@ API_Docs = "https://redocly.github.io/redoc/?url=https://actinia.mundialis.de/la`
`98`	`98`
`99`	`99`	`[tool.pytest.ini_options]`
`100`	`100`	`minversion = "6.0"`
`101`		`-addopts = "--cov actinia_core --cov-report term-missing --verbose --tb=line -x"`
	`101`	`+addopts = "--cov actinia_core --cov-report term-missing --verbose --tb=line -x -s"`
`102`	`102`	`testpaths = [`
`103`	`103`	`"tests",`
`104`	`104`	`]`
`105`	`105`	`markers = [`
`106`	`106`	`"dev: test current in development",`
`107`	`107`	`"unittest: completely independent test",`
	`108`	`+ "noauth: tests for actinia without authentication",`
`108`	`109`	`]`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`# performance processing of geographical data that uses GRASS GIS for`
`5`	`5`	`# computational tasks. For details, see https://actinia.mundialis.de/`
`6`	`6`	`#`
`7`		`-# Copyright (c) 2016-2018 Sören Gebbert and mundialis GmbH & Co. KG`
	`7`	`+# Copyright (c) 2016-2023 Sören Gebbert and mundialis GmbH & Co. KG`
`8`	`8`	`#`
`9`	`9`	`# This program is free software: you can redistribute it and/or modify`
`10`	`10`	`# it under the terms of the GNU General Public License as published by`
`@@ -116,9 +116,9 @@`
`116`	`116`	`from actinia_api import API_VERSION, URL_PREFIX`
`117`	`117`
`118`	`118`	`__license__ = "GPLv3"`
`119`		`-__author__ = "Sören Gebbert, Julia Haas"`
	`119`	`+__author__ = "Sören Gebbert, Julia Haas, Anika Weinmann"`
`120`	`120`	`__copyright__ = (`
`121`		`- "Copyright 2016-2021, Sören Gebbert and mundialis GmbH & Co. KG"`
	`121`	`+ "Copyright 2016-2023, Sören Gebbert and mundialis GmbH & Co. KG"`
`122`	`122`	`)`
`123`	`123`	`__maintainer__ = "mundialis"`
`124`	`124`
`@@ -165,11 +165,14 @@`
`165`	`165`	`}`
`166`	`166`	`}`
`167`	`167`	`]`
`168`		`-else:`
	`168`	`+elif global_config.AUTHENTICATION:`
`169`	`169`	`# Set the security definition in an unconventional way`
`170`	`170`	`flask_api._swagger_object["securityDefinitions"] = {`
`171`	`171`	`"basicAuth": {"type": "basic"}`
`172`	`172`	`}`
`173`	`173`	`flask_api._swagger_object["security"] = [{"basicAuth": []}]`
`174`	`174`
`175`	`175`	`auth = HTTPBasicAuth()`
	`176`	`+else:`
	`177`	`+ # No authentication`
	`178`	`+ auth = HTTPBasicAuth()`