Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JSON schema _strict_ must allow the property $schema #106

Closed
jkowalleck opened this issue Dec 19, 2021 · 0 comments · Fixed by #107
Closed

JSON schema _strict_ must allow the property $schema #106

jkowalleck opened this issue Dec 19, 2021 · 0 comments · Fixed by #107

Comments

@jkowalleck
Copy link
Member

current strict mode json schemas do not allow a root property $schema.
see CURRENT: 1.2 https://github.com/CycloneDX/specification/blob/e308a7da1ad1b72725d104d7dc2d2438214d65b0/schema/bom-1.3-strict.schema.json
see CURRENT: 1.3 https://github.com/CycloneDX/specification/blob/e308a7da1ad1b72725d104d7dc2d2438214d65b0/schema/bom-1.2-strict.schema.json

feature request: add the used schema as a property $schema of the resulting json.
the value must be the $id of the schema used.

spec schema-id
1.2 http://cyclonedx.org/schema/bom-1.2a.schema.json
1.3 http://cyclonedx.org/schema/bom-1.3.schema.json
@jkowalleck jkowalleck mentioned this issue Dec 19, 2021
Merged
jkowalleck added a commit to jkowalleck/fork_CycloneDX-specification that referenced this issue Dec 19, 2021
@jkowalleck
JSON strict: add optional root property $schema
36fcc24 
fixes CycloneDX#106

Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
This was referenced Dec 19, 2021
Merged
Closed
Closed
stevespringett added a commit that referenced this issue Dec 20, 2021
@stevespringett
#106 - Added $schema to 1.4-SNAPSHOT
344e22c 
Signed-off-by: Steve Springett <steve@springett.us>
stevespringett added a commit that referenced this issue Dec 20, 2021
@stevespringett
#106 - Added $schema to 1.4-SNAPSHOT
0ce5f5e 
Signed-off-by: Steve Springett <steve@springett.us>
@jkowalleck jkowalleck mentioned this issue Dec 20, 2021
Closed
@jkowalleck jkowalleck mentioned this issue Mar 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

JSON strict: add optional root property $schema jkowalleck/fork_CycloneDX-specification
1 participant
@jkowalleck