Commit 074436c0 authored by Dimitrios Gogos's avatar Dimitrios Gogos
Browse files

fix: clean up comments and docstrings for clarity and conciseness

parent 81405750

src/open_exposure_gateway/adapters/http/keycloak_token_verifier.py

+0 −5
Changes for src/open_exposure_gateway/adapters/http/keycloak_token_verifier.py: 0 added lines, 5 removed lines.
Original line number Diff line number Diff line
@@ -4,11 +4,6 @@ from open_exposure_gateway.core.config import KeycloakSettings 




class KeycloakTokenVerifier:

    """Adapter implementing TokenVerifierPort against Keycloak: the JWT

    verification logic itself lives in core/auth (JWKSCache + validate_token),

    this just gives callers a swappable seam -- a future auth mechanism only

    needs a new adapter here, not changes to dependencies.py/health.py."""



    def __init__(self, settings: KeycloakSettings) -> None:

        self._settings = settings

        self._jwks_cache = JWKSCache(settings.jwks_url)

src/open_exposure_gateway/api/camara/edge_application_management/v0_1_0_alpha_1/router.py

+0 −4
Changes for src/open_exposure_gateway/api/camara/edge_application_management/v0_1_0_alpha_1/router.py: 0 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -48,10 +48,6 @@ from open_exposure_gateway.schemas.common import ErrorInfo 
# by this package, {apiRoot} by the deployment.

BASE_PATH = "/edge-application-management/v0.1alpha1"



# require_role/require_three_legged/require_scope apply to every route in this router,

# looking each request's requirements up in the one explicit table

# (core/authorization.py) -- so authorization is enforced identically here without

# touching individual handlers.

router = APIRouter(

    prefix=BASE_PATH,

    dependencies=[Depends(require_role), Depends(require_three_legged), Depends(require_scope)],

src/open_exposure_gateway/api/camara/location_retrieval/v0_5_0/router.py

+0 −4
Changes for src/open_exposure_gateway/api/camara/location_retrieval/v0_5_0/router.py: 0 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -35,10 +35,6 @@ from open_exposure_gateway.schemas.common import ErrorInfo 
# v0.5.0), mounted bare per ADR-0020.

BASE_PATH = "/location-retrieval/v0.5"



# require_role/require_three_legged/require_scope apply to every route in this router,

# looking each request's requirements up in the one explicit table

# (core/authorization.py) -- so authorization is enforced identically here without

# touching individual handlers.

router = APIRouter(

    prefix=BASE_PATH,

    dependencies=[Depends(require_role), Depends(require_three_legged), Depends(require_scope)],

src/open_exposure_gateway/api/camara/quality_on_demand/v0_10_1/router.py

+0 −4
Changes for src/open_exposure_gateway/api/camara/quality_on_demand/v0_10_1/router.py: 0 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -31,10 +31,6 @@ from open_exposure_gateway.schemas.common import ErrorInfo 
# CAMARA base path: the spec serves at {apiRoot}/qod/v0 (wire version of v0.10.1).

BASE_PATH = "/qod/v0"



# require_role/require_three_legged/require_scope apply to every route in this router,

# looking each request's requirements up in the one explicit table

# (core/authorization.py) -- so authorization is enforced identically here without

# touching individual handlers.

router = APIRouter(

    prefix=BASE_PATH,

    dependencies=[Depends(require_role), Depends(require_three_legged), Depends(require_scope)],

src/open_exposure_gateway/application/mappers/edge_application_mapper.py

+1 −2
Changes for src/open_exposure_gateway/application/mappers/edge_application_mapper.py: 1 added line, 2 removed lines.
Original line number Diff line number Diff line
@@ -101,8 +101,7 @@ _CALLBACK_EVENT_SOURCE = "https://api.example.com/edge-application-management/v1 


_VISIBILITY_REVERSE_MAP: dict[str, str] = {v: k for k, v in _VISIBILITY_MAP.items()}



# Returned as `appProvider` for apps registered before the submitted name was stored in the

# spec descriptor. Must itself satisfy CAMARA's AppProvider pattern.

# For apps registered before `appProvider` was stored; must match CAMARA's AppProvider pattern.

UNKNOWN_APP_PROVIDER = "unknown_provider"



_SRM_STATE_TO_APP_INSTANCE_STATUS: dict[str, AppInstanceStatus] = {