-
Notifications
You must be signed in to change notification settings - Fork 333
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
GHSA-hgwp-4vp4-qmm2 has no published or compatible non-vulnerable versions #1742
Comments
As this refers to the Go ecosystem, it shoudl be something like fixed: I believe this was the tag it was fixed in: cloudflare/cloudflared@9323844 |
The version "2020.8.1" is not recognised as valid and therefore matches no revisions of the package. A "psuedo-version" will be matched correctly. https://go.dev/ref/mod#pseudo-versions Fixes github#1742 Signed-off-by: Thomas Way <[email protected]>
The version "2020.8.1" is not recognised as valid and therefore matches no revisions of the package. A "psuedo-version" will be matched correctly. https://go.dev/ref/mod#pseudo-versions Fixes github#1742 Signed-off-by: Thomas Way <[email protected]>
The version "2020.8.1" is not recognised as valid and therefore matches no revisions of the package. A "psuedo-version" will be matched correctly. https://go.dev/ref/mod#pseudo-versions Fixes github#1742 Signed-off-by: Thomas Way <[email protected]>
The version "2020.8.1" is not recognised as valid and therefore matches no revisions of the package. A "psuedo-version" will be matched correctly. https://go.dev/ref/mod#pseudo-versions Fixes github#1742 Signed-off-by: Thomas Way <[email protected]>
The version "2020.8.1" is not recognised as valid and therefore matches no revisions of the package. A "psuedo-version" will be matched correctly. https://go.dev/ref/mod#pseudo-versions Fixes github#1742 Signed-off-by: Thomas Way <[email protected]>
@uhthomas I just tried putting the pseudoversion in the advisory. What does the alert page look like now? |
Cool! The Dependabot warning is gone and Renovate has closed the incorrect security fix PR. uhthomas/renovate20706#4, replaced instead with a normal dependency update PR uhthomas/renovate20706#5. |
Though, Dependabot still isn't happy about something it seems.
It seems like some strange behaviour with how Dependabot finds dependency versions as Renovate seems able to manage. |
For go modules, Dependabot only supports updates using semver versions that are compatible with go modules. This is how we'd find the available versions:
It could be possible to allow Dependabot to try updating a later commit instead but may not be worth the effort to support non-conforming modules. I'm not sure this module is intended to be used as a dependency based on their versioning scheme. |
FWIW it is intended to be used as a dependency. We do so for some projects internally. I think we agree the versioning scheme is weird and I believe the authors are interested in using a more compliant semver scheme at some point. |
Just following up from the dependabot-updates end. We don't intend to support the non-conforming versioning scheme used by |
Pseudo versions like we provided to fix the issue should continue to work though, right? |
Yup, they'll work for Dependabot alerting & it's the correct way to list them in the advisory as that's how the versions are captured in the |
google/osv.dev#1084
The text was updated successfully, but these errors were encountered: