Skip to content

Commit 4e7bbbc

Browse files
committed
chore: add TODOs
1 parent e4651ee commit 4e7bbbc

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

  • src/aignostics_foundry_core/api

src/aignostics_foundry_core/api/auth.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
AUTH0_COOKIE_SCHEME_DESCRIPTION = "Auth0 session cookie authentication scheme."
2929
AUTH0_ROLE_ADMIN = "admin"
3030
USER_NOT_AUTHENTICATED = "User is not authenticated"
31+
# TODO(oliverm): remove the default; it should not reference Bridge
3132
DEFAULT_AUTH0_ROLE_CLAIM = "https://aignostics-platform-bridge/role"
3233

3334

@@ -40,8 +41,8 @@ class AuthSettings(OpaqueSettings):
4041

4142
model_config = SettingsConfigDict(extra="ignore")
4243

43-
internal_org_id: str | None = None
44-
auth0_role_claim: str = DEFAULT_AUTH0_ROLE_CLAIM
44+
internal_org_id: str | None = None # TODO(oliverm): make mandatory
45+
auth0_role_claim: str = DEFAULT_AUTH0_ROLE_CLAIM # TODO(oliverm): make mandatory and remove default
4546

4647
def __init__(self, **kwargs: Any) -> None: # noqa: ANN401
4748
"""Initialise settings, deriving env_prefix from the active FoundryContext."""

0 commit comments

Comments
 (0)