Skip to content
Snippets Groups Projects

fix: Fix error where GH pre-load crashes app if not data found

1 unresolved thread

Resolves #24 (closed)

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
159 159
160 160 return om.readerFor(CveProjectData.class).readValue(githubApi.getCveDetails(params));
161 161 } catch (Exception e) {
162 throw new RuntimeException(String.format("Could not fetch CVE %s from GitHub", key), e);
162 // Catch and log so as to not block the pre-load if GH has issues
163 LOGGER.error(String.format("Could not fetch CVE %s from GitHub", key), e);
  • added 1 commit

    • 354d9752 - feat: Add tests for CVE with no GH data

    Compare with previous version

  • Martin Lowe approved this merge request

    approved this merge request

  • merged

  • Martin Lowe mentioned in commit fc613ef9

    mentioned in commit fc613ef9

  • Please register or sign in to reply
    Loading