Skip to content

Commit 41a68b1

Browse files
Merge pull request #40 from FireTail-io/fix/keyerror-authz
added exception catch to authz
2 parents 47d5829 + 9fa7e46 commit 41a68b1

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

firetail/decorators/response.py

+10-5
Original file line numberDiff line numberDiff line change
@@ -115,11 +115,16 @@ def validate_response_authz(self, response_definition, data):
115115
return True
116116

117117
def extract_item(self, data, response_data_lookup):
118-
items = response_data_lookup.split(".")
119-
dc = data.copy()
120-
for i in items:
121-
dc = dc[i]
122-
return dc
118+
try:
119+
items = response_data_lookup.split(".")
120+
dc = data.copy()
121+
for i in items:
122+
dc = dc[i]
123+
return dc
124+
except KeyError:
125+
raise AuthzNotPopulated("Authz data does not contain expected key for authz to be evaluated")
126+
except Exception:
127+
raise AuthzFailed()
123128

124129
def is_json_schema_compatible(self, response_schema: dict) -> bool:
125130
"""

0 commit comments

Comments
 (0)