feat: Unify various API endpoints related to config

More details in the linked issue.

Closes #1960

backend/capellacollab/feedback/routes.py

@@ -9,12 +9,9 @@
 import fastapi
 from sqlalchemy import orm
 
-from capellacollab.config import config
 from capellacollab.core import database
 from capellacollab.core import logging as log
 from capellacollab.core.authentication import injectables as auth_injectables
-from capellacollab.settings.configuration import core as config_core
-from capellacollab.settings.configuration import models as config_models
 from capellacollab.users import injectables as user_injectables
 from capellacollab.users import models as users_models
 
@@ -23,19 +20,6 @@
 router = fastapi.APIRouter()
 
 
-@router.get(
-    "/configurations/feedback",
-    response_model=config_models.FeedbackConfiguration,
-)
-def get_feedback_configuration(
-    db: orm.Session = fastapi.Depends(database.get_db),
-):
-    feedback = config_core.get_global_configuration(db).feedback
-    if not (config.smtp and config.smtp.enabled):
-        util.disable_feedback(feedback)
-    return feedback
-
-
 @router.post(
     "/feedback",
     status_code=204,

backend/capellacollab/routes.py

@@ -11,8 +11,6 @@
 from capellacollab.events import routes as events_router
 from capellacollab.feedback import routes as feedback_routes
 from capellacollab.health import routes as health_routes
-from capellacollab.metadata import routes as core_metadata
-from capellacollab.navbar import routes as navbar_routes
 from capellacollab.notices import routes as notices_routes
 from capellacollab.projects import routes as projects_routes
 from capellacollab.sessions import routes as sessions_routes
@@ -30,8 +28,6 @@
     responses=auth_responses.api_exceptions(include_authentication=True),
     tags=["Health"],
 )
-router.include_router(core_metadata.router, tags=["Metadata"])
-router.include_router(navbar_routes.router, tags=["Navbar"])
 router.include_router(feedback_routes.router, tags=["Feedback"])
 router.include_router(
     sessions_routes.router,

backend/capellacollab/settings/configuration/routes.py

@@ -9,28 +9,33 @@
 from capellacollab.core import database
 from capellacollab.core.authentication import injectables as auth_injectables
 from capellacollab.feedback import util as feedback_util
+from capellacollab.settings.configuration.unifiedconfig import (
+    routes as unified_config_routes,
+)
 from capellacollab.users import crud as users_crud
 from capellacollab.users import models as users_models
 
 from . import core, crud, models
 
 router = fastapi.APIRouter(
-    dependencies=[
-        fastapi.Depends(
-            auth_injectables.RoleVerification(
-                required_role=users_models.Role.ADMIN
-            )
-        )
-    ],
     tags=["Configuration"],
 )
 
+router.include_router(unified_config_routes.router, tags=["Unified Config"])
+
 schema_router = fastapi.APIRouter(dependencies=[], tags=["Configuration"])
 
 
 @router.get(
     f"/{models.GlobalConfiguration._name}",
     response_model=models.GlobalConfiguration,
+    dependencies=[
+        fastapi.Depends(
+            auth_injectables.RoleVerification(
+                required_role=users_models.Role.ADMIN
+            )
+        )
+    ],
 )
 async def get_configuration(
     db: orm.Session = fastapi.Depends(database.get_db),
@@ -41,6 +46,13 @@ async def get_configuration(
 @router.put(
     f"/{models.GlobalConfiguration._name}",
     response_model=models.GlobalConfiguration,
+    dependencies=[
+        fastapi.Depends(
+            auth_injectables.RoleVerification(
+                required_role=users_models.Role.ADMIN
+            )
+        )
+    ],
 )
 async def update_configuration(
     body: models.GlobalConfiguration,

backend/capellacollab/settings/configuration/unifiedconfig/__init__.py

@@ -0,0 +1,2 @@
+# SPDX-FileCopyrightText: Copyright DB InfraGO AG and contributors
+# SPDX-License-Identifier: Apache-2.0

backend/capellacollab/settings/configuration/unifiedconfig/models.py

@@ -0,0 +1,25 @@
+# SPDX-FileCopyrightText: Copyright DB InfraGO AG and contributors
+# SPDX-License-Identifier: Apache-2.0
+
+from capellacollab.core import pydantic as core_pydantic
+from capellacollab.settings.configuration import models as config_models
+
+
+class Metadata(core_pydantic.BaseModel):
+    version: str
+    privacy_policy_url: str | None
+    imprint_url: str | None
+    provider: str | None
+    authentication_provider: str | None
+    environment: str | None
+
+    host: str | None
+    port: str | None
+    protocol: str | None
+
+
+class UnifiedConfig(core_pydantic.BaseModel):
+    metadata: Metadata
+    feedback: config_models.FeedbackConfiguration
+    navbar: config_models.NavbarConfiguration
+    beta: config_models.BetaConfiguration

backend/capellacollab/settings/configuration/unifiedconfig/routes.py

@@ -0,0 +1,27 @@
+# SPDX-FileCopyrightText: Copyright DB InfraGO AG and contributors
+# SPDX-License-Identifier: Apache-2.0
+
+import fastapi
+from sqlalchemy import orm
+
+from capellacollab.core import database
+from capellacollab.settings.configuration import core as config_core
+from capellacollab.settings.configuration.unifiedconfig import models, util
+
+router = fastapi.APIRouter()
+
+
+@router.get(
+    "/unified",
+)
+def get_unified_config(
+    db: orm.Session = fastapi.Depends(database.get_db),
+) -> models.UnifiedConfig:
+    cfg = config_core.get_global_configuration(db)
+
+    return models.UnifiedConfig(
+        metadata=util.get_metadata(cfg),
+        feedback=util.get_feedback(cfg),
+        beta=util.get_beta(cfg),
+        navbar=util.get_navbar(cfg),
+    )

backend/capellacollab/settings/configuration/unifiedconfig/util.py

@@ -0,0 +1,47 @@
+# SPDX-FileCopyrightText: Copyright DB InfraGO AG and contributors
+# SPDX-License-Identifier: Apache-2.0
+
+import capellacollab
+from capellacollab.config import config
+from capellacollab.settings.configuration import models as config_models
+from capellacollab.settings.configuration.unifiedconfig import models
+
+
+def get_metadata(
+    global_config: config_models.GlobalConfiguration,
+) -> models.Metadata:
+    return models.Metadata.model_validate(
+        global_config.metadata.model_dump()
+        | {
+            "version": capellacollab.__version__,
+            "host": config.general.host,
+            "port": str(config.general.port),
+            "protocol": config.general.scheme,
+        }
+    )
+
+
+def get_feedback(
+    global_config: config_models.GlobalConfiguration,
+) -> config_models.FeedbackConfiguration:
+    feedback = global_config.feedback
+    if not (config.smtp and config.smtp.enabled):
+        feedback.enabled = False
+        feedback.after_session = False
+        feedback.on_footer = False
+        feedback.on_session_card = False
+        feedback.interval.enabled = False
+
+    return feedback
+
+
+def get_navbar(
+    global_config: config_models.GlobalConfiguration,
+) -> config_models.NavbarConfiguration:
+    return global_config.navbar
+
+
+def get_beta(
+    global_config: config_models.GlobalConfiguration,
+) -> config_models.BetaConfiguration:
+    return global_config.beta

backend/capellacollab/users/routes.py

@@ -16,7 +16,6 @@
 from capellacollab.projects.users import crud as projects_users_crud
 from capellacollab.sessions import routes as session_routes
 from capellacollab.settings.configuration import core as config_core
-from capellacollab.settings.configuration import models as config_models
 from capellacollab.users import injectables as users_injectables
 from capellacollab.users import models as users_models
 from capellacollab.users.tokens import routes as tokens_routes
@@ -41,18 +40,6 @@ def get_current_user(
     return user
 
 
-@router.get(
-    "/beta",
-    response_model=config_models.BetaConfiguration,
-)
-def get_beta_config(db: orm.Session = fastapi.Depends(database.get_db)):
-    cfg = config_core.get_global_configuration(db)
-
-    return config_models.BetaConfiguration.model_validate(
-        cfg.beta.model_dump()
-    )
-
-
 @router.get(
     "/{user_id}",
     response_model=models.User,

backend/tests/settings/test_global_configuration.py

@@ -128,9 +128,9 @@ def test_metadata_is_updated(
 
     assert response.status_code == 200
 
-    response = client.get("/api/v1/metadata")
+    response = client.get("/api/v1/settings/configurations/unified")
     assert response.status_code == 200
-    assert response.json()["environment"] == "test"
+    assert response.json()["metadata"]["environment"] == "test"
 
 
 @pytest.mark.usefixtures("admin")
@@ -154,9 +154,9 @@ def test_navbar_is_updated(
 
     assert response.status_code == 200
 
-    response = client.get("/api/v1/navbar")
+    response = client.get("/api/v1/settings/configurations/unified")
     assert response.status_code == 200
-    assert response.json()["external_links"][0] == {
+    assert response.json()["navbar"]["external_links"][0] == {
         "name": "Example",
         "href": "https://example.com",
         "role": "user",

backend/tests/test_feedback.py

@@ -210,9 +210,9 @@ def test_feedback_is_updated(
 
     assert response.status_code == 200
 
-    response = client.get("/api/v1/configurations/feedback")
+    response = client.get("/api/v1/settings/configurations/unified")
     assert response.status_code == 200
-    assert response.json() == {
+    assert response.json()["feedback"] == {
         "enabled": True,
         "after_session": True,
         "on_footer": True,
@@ -262,9 +262,9 @@ def test_activate_feedback_without_recipients(
 
 @pytest.mark.usefixtures("user", "smtp_config_not_set", "feedback_enabled")
 def test_feedback_is_disabled_without_smtp(client: testclient.TestClient):
-    response = client.get("/api/v1/configurations/feedback")
+    response = client.get("/api/v1/settings/configurations/unified")
     assert response.status_code == 200
-    assert response.json()["enabled"] is False
+    assert response.json()["feedback"]["enabled"] is False
 
 
 def test_feedback_metric(db: orm.Session):