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

feat: improve error reporting for reachable vulnerabilities #1314

Merged
merged 2 commits into from
Aug 7, 2020

Conversation

muscar
Copy link
Contributor

@muscar muscar commented Aug 7, 2020

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Bump the Maven plugin version, and make some small adjustments to accommodate the changes in error reporting introduced in the latest version.

Also fixes --reachable behaviour on Windows

How should this be manually tested?

Run snyk test -d --reachable --reachable-timeout=5, and check the error reported for call graph generation.

What are the relevant tickets?

https://snyksec.atlassian.net/browse/FLOW-373

Screenshots

image

@muscar muscar requested review from a team as code owners August 7, 2020 11:19
@ghost ghost requested review from anthogez and gitphill August 7, 2020 11:19
--reachable flag wasn't working correctly on windows machines, this is the transitive update of a dependency with a fix
@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2020

Expected release notes (by @miiila)

features:
improve error reporting for reachable vulnerabilities (31415a9)

fixes:
--reachable on Windows (0d2d9e7)

others (will not be included in Semantic-Release notes):
e02e527 chore:updated gitignore with vscode folder

  • I hereby acknowledge these release notes are 🥙 AWESOME 🥙

@miiila miiila merged commit c46a4c8 into master Aug 7, 2020
@miiila miiila deleted the chore/flow-373-rework-error-reporting branch August 7, 2020 14:46
@snyksec
Copy link

snyksec commented Aug 7, 2020

🎉 This PR is included in version 1.370.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants