oasis-tcs / cti-stix2

OASIS CTI TC: Provides issue tracking and wiki pages for the STIX 2.x Work Products
https://github.com/oasis-tcs/cti-stix2
Other
24 stars 9 forks source link

Should we change the normative statement about extension_types/extension_properties properties #259

Open rpiazza opened 3 years ago

rpiazza commented 3 years ago

The spec says in section 7.3

This property (extension_properties) MUST only be used when the extension_types property includes a value of toplevel-property-extension

But then also says:

When this property includes toplevel-property-extension then the extension_properties property SHOULD include one or more property names.

Why isn't this a MUST. It would help immensely for validation

allant0 commented 3 years ago

Because there are cases where the schema isn't directly in the object itself and therefore validation should be done outside of the spec language if you really want to do validation.

allant0 commented 3 years ago

And another reason. This is a material change not worth changing the spec for. So imo its a mute point. It shouldn't be done ever and certainly not for this spec version.

jordan2175 commented 3 years ago

We talked about this and will talk about it again next week. The up down vote was split. People on the call wanted a bit more time to think about it.

jordan2175 commented 3 years ago

Talked about this on the working call, and we decided to push to a future release.