microsoft/sarif-sdk

[REQUEST] Allow users of the SARIF SDK to emit SARIF results with both ruleId and rule.id

dotpaul opened this issue · 0 comments

The ShouldSerializeRuleId() method in

public bool ShouldSerializeRuleId()
disallows serializing ruleId when rule.id is present.

For consumers of the SARIF SDK who wish to emit both ruleId and rule.id (I imagine there a few implementations of reading SARIF files only expecting ruleId?), it'd be nice if the ShouldSerializeRuleId() method was virtual. Then a consumer could derive from the Result class and override to always serialize the ruleId property