Skip to content
Snippets Groups Projects

fix: Fix issue deserializing GH CVE object

Merged fix: Fix issue deserializing GH CVE object
zacharysabourin/eclipsefdn-cve-api:zacharysabourin/main/27 into main
1 unresolved thread
Merged Zachary Sabourin requested to merge zacharysabourin/eclipsefdn-cve-api:zacharysabourin/main/27 into main
1 unresolved thread

Resolves #27 (closed)

The augmented CVE data from GH wasn't deserializing properly so the description and cvss fields weren't be populated

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
src/main/java/org/eclipsefoundation/cve/service/CveService.java
116 116 .orElse(LocalizedValue.builder().setLang("en").setValue("").build())
117 117 .getValue())
118 118 .setCvss(cveDetails.getImpact().isPresent()
119 ? cveDetails.getImpact().get().getCvss().getBaseScore()
119 ? cveDetails.getImpact().get().getCvss().get(0).getBaseScore()
  • Zachary Sabourin added 1 commit

    added 1 commit

    • 281b1b92 - fix: Fix cvss impact access validation

    Compare with previous version

  • Zachary Sabourin added 1 commit

    added 1 commit

    • 2487fe87 - Fix logic operator in cveDetails

    Compare with previous version

  • Martin Lowe approved this merge request

    approved this merge request

  • Martin Lowe mentioned in commit 08e3cf80

    mentioned in commit 08e3cf80

  • Martin Lowe merged

    merged

    • Please register or sign in to reply

    Copyright © Eclipse Foundation, Inc. All Rights Reserved.     Privacy Policy | Terms of Use | Copyright Agent