Replies: 1 comment
-
|
Similar thoughts here The design seems bad, or perhaps it's deliberately intended to discourage 3rd party usage Why the vendor enums - no one else gets to be a vendor? Same with dialects. Why custom extensions have a value as a JSON string embedded in the YAML doc? Extension ids could be URIs, domain name would prevent collisions Why even group things under |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I think the current definition for
custom_extensionsis overly prescribed when it doesn’t need to be. The intent of this field seems to be to encode any additional data that isn’t native to the OSI, such that other systems (or a re-comsumption by the original system) may opportunistically read.Currently it fails at this task, since
vendor_nameis a required enum that is currently very limited to 3 big players and “COMMON” which gives a single namespace for everyone else. Thisvendor_nameapproach also gives little room for a single vendor to produce multiple extensions. The data encoding is also encoded as a JSON string for no practical reason.Particularly as application authors do novel things, this means that either A) application authors will break the specification by encoding things that technically aren’t allowed by the specification format, or B) applications will not be able to use OSI for novel things without first pushing the entirety of their use case through extensive review.
I propose a simpler definition for this property:
Where
extension_idis a vendor-specific prefixed bundle identifier.This allows each vendor to contribute and read against this field without any constraints, while maintaining a decent pattern to not step on one another:
Beta Was this translation helpful? Give feedback.
All reactions