-
Notifications
You must be signed in to change notification settings - Fork 76
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
Subpackages with different versions are incorrectly flagged #270
Comments
Just in case it helps: Have you tried invoking |
here you are:
|
same for
|
Could you verify if this is still an issue with the latest version of Nancy? I ask because the original report mentions it being fixed in a newer version. |
nancy --version
nancy version 1.0.46 |
Same issue with current version. The
|
Same for github.com/hashicorp/consul/sdk@v0.16.1
|
Thanks for creating an issue! Please fill out this form so we can be
sure to have all the information we need, and to minimize back and forth.
The issue is already reported here Subpackages with different versions are incorrectly flagged #157, but this time with the package github.com/hashicorp/vault.
Releases are tagged here: https://github.com/hashicorp/vault/releases the latest version is v1.11.3
The latest version of github.com/hashicorp/vault/api is v1.7.2
The latest version of github.com/hashicorp/vault/sdk is v0.5.3
Nancy sees I have used github.com/hashicorp/vault/sdk@v0.5.1 and but reports CVE's as if I was using github.com/hashicorp/vault@v0.5.1 same for github.com/hashicorp/vault/api@v1.7.2 - is recommended to be github.com/hashicorp/vault@1.11.1/1.9.8/1.10.5
None of these CVE's should apply to github.com/hashicorp/vault/api@v1.7.2 or github.com/hashicorp/vault/sdk@v0.5.1 but it gets confused by the version difference.
Here is what I get:
Here is the output of go list:
Here is my Nancy version:
nancy version 1.0.39
What feature or behavior is this required for?
Go dependencies vulnerability scan
How could we solve this issue? (Not knowing is okay!)
I think if a subpackage is versioned differently it should be considered separate and not matched against the parent one?
Anything else?
No
cc @bhamail / @DarthHater
The text was updated successfully, but these errors were encountered: