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

transformer: handle null in src.url during deserialization #131

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

secufoe
Copy link
Contributor

@secufoe secufoe commented Sep 7, 2024

Turns out that nix can produce a src field only containing "url": null. This leads to:

> Error: Failed to parse buildtime input
>
> Caused by:
>     invalid type: null, expected a string at line 335292 column 17

@secufoe
Copy link
Contributor Author

secufoe commented Sep 7, 2024

I guess CycloneDX doesn't allow an empty URL: https://cyclonedx.org/docs/1.4/json/#externalReferences_items_url

@secufoe
Copy link
Contributor Author

secufoe commented Sep 7, 2024

@secufoe secufoe force-pushed the src-url-deserializer branch from 952d8f3 to 9a5845c Compare September 9, 2024 08:15
@secufoe secufoe force-pushed the src-url-deserializer branch from 9a5845c to fcadf9b Compare September 9, 2024 08:29
Copy link
Owner

@nikstur nikstur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nikstur nikstur merged commit d5a20d0 into nikstur:main Sep 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants