Skip to content

Commit dfe780c

Browse files
committed
remove redundant allowed
1 parent 67e5a32 commit dfe780c

File tree

1 file changed

+0
-45
lines changed

1 file changed

+0
-45
lines changed

horizon/enforcer/api.py

Lines changed: 0 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -555,51 +555,6 @@ async def is_allowed(
555555
)
556556
return result
557557

558-
@router.post(
559-
"/allowed",
560-
response_model=AuthorizationResult,
561-
status_code=status.HTTP_200_OK,
562-
response_model_exclude_none=True,
563-
dependencies=[Depends(enforce_pdp_token)],
564-
)
565-
async def is_allowed(
566-
request: Request,
567-
query: Union[AuthorizationQuery, AuthorizationQueryV1],
568-
x_permit_sdk_language: Optional[str] = Depends(notify_seen_sdk),
569-
):
570-
if isinstance(query, AuthorizationQueryV1):
571-
raise HTTPException(
572-
status_code=status.HTTP_421_MISDIRECTED_REQUEST,
573-
detail="Mismatch between client version and PDP version,"
574-
" required v2 request body, got v1. "
575-
"hint: try to update your client version to v2",
576-
)
577-
query = cast(AuthorizationQuery, query)
578-
579-
response = await _is_allowed(query, request, MAIN_POLICY_PACKAGE)
580-
log_query_result(query, response)
581-
try:
582-
raw_result = json.loads(response.body).get("result", {})
583-
processed_query = (
584-
get_v1_processed_query(raw_result)
585-
or get_v2_processed_query(raw_result)
586-
or {}
587-
)
588-
result = {
589-
"allow": raw_result.get("allow", False),
590-
"result": raw_result.get(
591-
"allow", False
592-
), # fallback for older sdks (TODO: remove)
593-
"query": processed_query,
594-
"debug": raw_result.get("debug", {}),
595-
}
596-
except:
597-
result = dict(allow=False, result=False)
598-
logger.warning(
599-
"is allowed (fallback response)", reason="cannot decode opa response"
600-
)
601-
return result
602-
603558
@router.post(
604559
"/nginx_allowed",
605560
response_model=AuthorizationResult,

0 commit comments

Comments
 (0)