Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider SARIF support for tool output -> MSBuild logs #8028

Open
rainersigwald opened this issue Oct 6, 2022 · 2 comments
Open

Consider SARIF support for tool output -> MSBuild logs #8028

rainersigwald opened this issue Oct 6, 2022 · 2 comments

Comments

@rainersigwald
Copy link
Member

The Microsoft C++ team is considering adding SARIF support to the MSVC toolchain. That allows capturing much more information about errors/warnings than we currently support, through the canonical error format.

MSBuild should consider extending the tool-output-scraping support to include SARIF output, and map the SARIF format to our error/warning structured events.

This might also help with issues like #8019 that primarily arise because an error cannot today have supplemental information.

@rainersigwald rainersigwald added Feature Request Area: Logging needs-triage Have yet to determine what bucket this goes in. labels Oct 6, 2022
@rainersigwald
Copy link
Member Author

One thing that might be important: today's output scanners are line-at-a-time, so this might require tool output SARIF to be in JSON lines.

@benvillalobos benvillalobos removed the needs-triage Have yet to determine what bucket this goes in. label Oct 6, 2022
@rainersigwald
Copy link
Member Author

@yuehuang010 is working on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants