Skip to content

fix(vulnfeeds): unmarshalling error in override upload + tests#4975

Merged
jess-lowe merged 4 commits intogoogle:masterfrom
jess-lowe:fix/override-upload
Mar 6, 2026
Merged

fix(vulnfeeds): unmarshalling error in override upload + tests#4975
jess-lowe merged 4 commits intogoogle:masterfrom
jess-lowe:fix/override-upload

Conversation

@jess-lowe
Copy link
Copy Markdown
Contributor

Thanks to the new logger errors, we discovered that the override upload is not using the right marshaller.

This PR fixes that, and adds some tests for mocking uploads to GCS.

michaelkedar
michaelkedar previously approved these changes Mar 5, 2026
Comment thread vulnfeeds/upload/cveworker_test.go
@jess-lowe jess-lowe merged commit 8d6fc14 into google:master Mar 6, 2026
21 checks passed
@jess-lowe jess-lowe deleted the fix/override-upload branch March 6, 2026 04:01
tymzd pushed a commit to tymzd/osv.dev that referenced this pull request Apr 13, 2026
…e#4975)

Thanks to the new logger errors, we discovered that the override upload
is not using the right marshaller.

This PR fixes that, and adds some tests for mocking uploads to GCS.
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