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

Bugfix - Cve should have an empty constructor #644

Merged
merged 1 commit into from
Apr 19, 2022
Merged

Bugfix - Cve should have an empty constructor #644

merged 1 commit into from
Apr 19, 2022

Conversation

yahavi
Copy link
Member

@yahavi yahavi commented Apr 19, 2022

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

The Cve class is used by the JFrog IDEA plugin's cache. An attempt to deserialize a cache object will fail with the following error:

com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Cannot construct instance of org.jfrog.build.extractor.scan.Cve (no Creators, like default constructor, exist): cannot deserialize from Object value (no delegate- or property-based Creator)

@yahavi yahavi requested a review from eyalbe4 April 19, 2022 06:01
@yahavi yahavi self-assigned this Apr 19, 2022
@yahavi yahavi merged commit 0b8deb2 into jfrog:master Apr 19, 2022
@yahavi yahavi deleted the cve-constructor branch April 19, 2022 06:11
@yahavi yahavi mentioned this pull request May 26, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyalbe4 eyalbe4 eyalbe4 approved these changes

Assignees

@yahavi yahavi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yahavi @eyalbe4