OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.


Help: OASIS Mailing Lists Help | MarkMail Help

sarif message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]

Subject: RE: Change draft for #267: Allow multiple properties in external files

The naming convention discussion continues:


Michael proposes:

  • externalFile => externalPropertyFile
  • externalizedProperty => sarifExternalProperties
  • run.externalFiles => run.externalPropertyFiles


And then the extension becomes .sarif-external-properties.


I am fine with this.


From: sarif@lists.oasis-open.org <sarif@lists.oasis-open.org> On Behalf Of Larry Golding (Myriad Consulting Inc)
Sent: Tuesday, October 23, 2018 2:15 PM
To: OASIS SARIF TC Discussion List <sarif@lists.oasis-open.org>
Subject: [sarif] Change draft for #267: Allow multiple properties in external files
Importance: High


I pushed a change draft for Issue #267, “Allow multiple properties in external files”:




We will move its adoption at tomorrow’s TC meeting.


This comes from Michael’s idea: you might have a huge results array, but the rest of the externalizable properties might be small. In that case you might want to keep the results array in the root file, and externalize all the other properties to a single external file. (Of course, the alternative would be to externalize the results array and leave the rest of the properties in the root file.)


While writing this I came up with what I think is a better set of names for some of the objects and properties, as follows:


  • Rename the externalFile object to propertyFile.
  • Rename the externalizedProperty object to sarifProperties.
  • In the run object:
    • Rename the externalFiles property to propertyFiles.


I came to this conclusion after searching for the word “external” in the spec and realizing that we use the term “external file” generically, to refer to any file that’s not part of the SARIF log file. So using “external file” to specifically mean a file that contains external properties would be confusing.


This in turn feeds into my suggestions for filename extensions (Issue #118), which is, by design, on the agenda immediately after #267.




[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]