OASIS Static Analysis Results Interchange Format (SARIF) TC

Erratum in eballot proposal for Issue #321

  • 1.  Erratum in eballot proposal for Issue #321

    Posted 03-07-2019 00:28
    I added the following POST-BALLOT NOTE to the eballot proposal comment for Issue #321 , “Provide mechanism for inlining externalized properties data into the root log.”   Erratum: The name of the new property should be  sarifLog.inlineExternalProperties , not  inlineExternalPropertyFiles , and it is an array of  externalProperties  objects (as defined in §4.3 of the spec), not  externalPropertyFile  objects.   If you look at all the examples above, you'll see that they really are arrays of  externalProperties objects.  @kupsch 's first example proposed to name the new property  inlineExternalFiles , which would be ok. Later in the thread, I proposed  inlineExternalProperties , as suggested here, which I prefer because it matches the name of the objects in the array.   " inlineExternalPropertyFiles ", as we proposed in the ballot, is just wrong. There  is  an object in the spec called  externalPropertyFile , but it is just a "locator" for an external file. Our new  sarifLog property contains the  contents  of "external" files, not locators for them.   I do not expect this to change anyone’s vote. This really is essentially a typo, as you can see by looking at the examples further up the issue’s discussion thread.   Thanks, Larry