search

microsoft / sarif-sdk

.NET code and supporting files for working with the 'Static Analysis Results Interchange Format' (SARIF, see https://github.com/oasis-tcs/sarif-spec)
Other
191 stars 88 forks source link

Use new 'PeekableStream' to do binary categorization. #2753

Closed scottoneil-ms closed 6 months ago

scottoneil-ms commented 6 months ago

PeekableStream offers a very weak Seek(Origin) capability that expires once a configurable amount of Stream has been read.

michaelcfanning commented 6 months ago

Release note, please! For this excellent change.

michaelcfanning commented 6 months ago

I know the class is internal and we're swapping out for function that should be equivalent. We'll want signal in the release notes of when we made the switch-over though, in case users report an issue in the enumerated artifact class.

In reply to: 1885935208