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.

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

fix: ProtoBuf evidence not repeated, but optional #425

Conversation

jkowalleck
Copy link
Member

@jkowalleck jkowalleck commented Apr 10, 2024

fixes #422
and yes, fixing this bug is actually considered a breaking change in terms of ProtoBuf

@stevespringett stevespringett merged commit 51e8197 into CycloneDX:master Aug 31, 2024
7 of 8 checks passed
@jkowalleck
Copy link
Member Author

@stevespringett, I think this merge should be reverted.
it introduced breaking changes.

tools generating ProtBuff messages for the current stable release 1.6 are incompatible to the current unstable(master) version since this merge.

this would also reopen #422 .
I am unsure about a non-breaking "fix" . we could create one by annotating that the "wrong" repeated should be used once at max, and that ingesting tools should ignore every other list item ...

jkowalleck added a commit to jkowalleck/fork_CycloneDX-specification that referenced this pull request Sep 4, 2024
Signed-off-by: Jan Kowalleck <[email protected]>
@jkowalleck jkowalleck mentioned this pull request Sep 4, 2024
jkowalleck added a commit that referenced this pull request Sep 4, 2024
This PR reverts previously wrong merge of PR #425 (which introduced
unexpected breaking-changes)
See
#425 (comment)


reopen #422

Signed-off-by: Jan Kowalleck <[email protected]>
jkowalleck added a commit that referenced this pull request Sep 4, 2024
…ated (#517)

non-breaking fix of #422 
in contrast to #425

Signed-off-by: Jan Kowalleck <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

component.evidence is repeated in proto and object in jsonschema
2 participants