Skip to content

Commit 31ad93c

Browse files
committed
Use starlette statuses directly
1 parent 9d8c30b commit 31ad93c

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/blueapi/service/authorization.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
from aiohttp import ClientSession
88
from fastapi import Depends, HTTPException, Request
9-
from starlette import status
9+
from starlette.status import HTTP_401_UNAUTHORIZED, HTTP_403_FORBIDDEN
1010

1111
from blueapi.config import OIDCConfig, OpaConfig, ServiceAccount
1212
from blueapi.service.authentication import TiledAuth, unchecked_bearer_token
@@ -73,7 +73,7 @@ async def require_submit_task(self, instrument_session: str, token: str):
7373
"visit": int(match["visit"]),
7474
},
7575
):
76-
raise HTTPException(status_code=status.HTTP_403_UNORTHORIZED)
76+
raise HTTPException(status_code=HTTP_403_FORBIDDEN)
7777

7878
async def is_admin(self, token: str) -> bool:
7979
return await self._call_opa(self._conf.admin_check, {"token": token})
@@ -118,13 +118,13 @@ async def opa(
118118

119119
if opa := cast(OpaClient | None, getattr(request.app.state, "authz", None)):
120120
if not token:
121-
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED)
121+
raise HTTPException(status_code=HTTP_401_UNAUTHORIZED)
122122
return OpaUserClient(opa, token)
123123
return None
124124

125125

126126
async def submit_permission(
127-
opa: Annotated[OpaUserClient, Depends(opa)],
127+
opa: Annotated[OpaUserClient | None, Depends(opa)],
128128
task_request: TaskRequest,
129129
):
130130
if opa:

0 commit comments

Comments
 (0)