Skip to content

Commit edce50f

Browse files
committed
Add missing result[] handling
fixes missing result errors #+BEGIN_SRC text Traceback (most recent call last): File "/mnt/common/home/hohn/work-gh/sarif-cli/.venv-ubuserv/bin/sarif-to-table", line 125, in <module> num_results = len(S.get(sarif_struct, 'runs', runi, 'results')) File "/mnt/common/home/hohn/work-gh/sarif-cli/sarif_cli/traverse.py", line 169, in get res = res[p] KeyError: 'results' 76% 3204:1006=7s ./repos/RasaHQ/rasa/code-scanning/analyses/132221999.sarif Traceback (most recent call last): File "/mnt/common/home/hohn/work-gh/sarif-cli/.venv-ubuserv/bin/sarif-to-table", line 125, in <module> num_results = len(S.get(sarif_struct, 'runs', runi, 'results')) File "/mnt/common/home/hohn/work-gh/sarif-cli/sarif_cli/traverse.py", line 169, in get res = res[p] KeyError: 'results' #+END_SRC
1 parent 1909517 commit edce50f

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

bin/sarif-to-table

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,13 @@ atexit.register(_flush_buffer)
122122
# --------------------------------------------------------------------
123123
for runi in S.indices(sarif_struct, 'runs'):
124124
unique_id = compute_unique_id(sarif_struct, runi, args.file)
125-
num_results = len(S.get(sarif_struct, 'runs', runi, 'results'))
126-
if num_results == 0:
125+
run_obj = S.get(sarif_struct, 'runs', runi)
126+
results = run_obj.get('results', [])
127+
if not results:
128+
S.dbg(f"Skipping {args.file} run {runi}: no results key\n")
127129
continue
130+
num_results = len(results)
131+
128132
for resi in S.indices(sarif_struct, 'runs', runi, 'results'):
129133
result = S.get(sarif_struct, 'runs', runi, 'results', resi)
130134
rule_id = result.get("ruleId")

0 commit comments

Comments
 (0)