alexedwards / argon2id

Argon2id password hashing and verification for Go
MIT License
452 stars 44 forks source link

golang.org/x/crypto dep should uptdae #2

Closed buffge closed 2 years ago

buffge commented 4 years ago

Vulnerabilities

DepShield reports that this application's usage of golang.org/x:crypto:0.0.0-20190308221718-c2843e01d9a2 results in the following vulnerability(s):

alexedwards commented 4 years ago

Thanks for this.

This package has golang.org/x/crypto v0.0.0-20190605123033 as a dependency, so I thought that it was a bit strange that this was being flagged.

But it looks like golang.org/x/crypto v0.0.0-20190605123033 has golang.org/x/net v0.0.0-20190404232315 a dependency, which in turn has golang.org/x/crypto v0.0.0-20190308221718 as dependency (see here). And that's the source of the warning.

So the fix for this is that the golang.org/x/net dep needs to be upgraded.

I'll keep this open until that happens.

But it looks like the vuln relates to the golang.org/x/crypto/salsa20 package only, which isn't used by argon2id, so this package isn't actually affected anyway.

buffge commented 4 years ago

l use the sonatype-depshield app ,this issues is it prompt me. You can try it. Thanks

alexedwards commented 2 years ago

The current version of golang.org/x/crypto now uses a newer version of golang.org/x/net, and I've upgraded the go.mod accordingly so this can finally be closed :champagne: