microsoft / sarif-visualstudio-extension

SARIF Microsoft Visual Studio Viewer Extension
MIT License
46 stars 19 forks source link

Fix issue cannot open a SARIF log if there is any tool notification. #608

Closed yongyan-gh closed 1 year ago

yongyan-gh commented 1 year ago

If the SARIF log contains tool notifications (which is not a SARIF result), viewer got unhandled exception and not able to load the results.

Attached the repro SARIF log. repros.zip

The fix is to skip the notifications in new added code finder process.

yongyan-gh commented 1 year ago

Bug 2116159: CRITICAL: vs viewer won't open SARIF