Skip to content

Commit 56b2150

Browse files
NPA-4599: Update python scripts to check path suffix contains rather than equals
1 parent 7384eae commit 56b2150

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
path_suffix = flow.getVariable("proxy.pathsuffix").lower()
22
request_verb = flow.getVariable("request.verb").lower()
33

4-
requested_endpoint = (path_suffix, request_verb)
5-
6-
7-
auth_forbidden = requested_endpoint in [
4+
blocked_resources = [
85
("/fhir/r4/relatedperson", "get"),
96
("/fhir/r4/questionnaireresponse", "post"),
107
]
118

9+
for blocked_resources in blocked_resources:
10+
if blocked_resources[0] in path_suffix and blocked_resources[1] == request_verb:
11+
auth_forbidden = True
12+
1213
flow.setVariable("app_auth_forbidden", auth_forbidden)

proxies/live/apiproxy/resources/py/check-user-enabled-endpoint.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@
22
path_suffix = flow.getVariable("proxy.pathsuffix").lower()
33
request_verb = flow.getVariable("request.verb").lower()
44

5-
requested_resource = (path_suffix, request_verb)
6-
75
if auth_level == "p9":
86
blocked_resources = [("/fhir/r4/consent", "post"), ("/fhir/r4/consent", "patch")]
97
elif auth_level == "all3":
108
blocked_resources = [("/fhir/r4/questionnaireresponse", "post")]
119
else:
1210
blocked_resources = []
1311

14-
auth_forbidden = requested_resource in blocked_resources
12+
for blocked_resources in blocked_resources:
13+
if blocked_resources[0] in path_suffix and blocked_resources[1] == request_verb:
14+
auth_forbidden = True
1515

1616
flow.setVariable("user_auth_forbidden", auth_forbidden)

0 commit comments

Comments
 (0)