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

Enhance the security of dependencies #24

Closed
jiayouxujin opened this issue May 15, 2023 · 2 comments
Closed

Enhance the security of dependencies #24

jiayouxujin opened this issue May 15, 2023 · 2 comments
Assignees

Comments

@jiayouxujin
Copy link
Collaborator

Hi,
Considering the security concerns with the dependency github.com/satori/go.uuid, we should replace it.

@tianshimoyi
Copy link
Collaborator

@jiayouxujin
Does github.com/satori/go.uuid have any security implications? Can you describe it?

@jiayouxujin
Copy link
Collaborator Author

@jiayouxujin Does github.com/satori/go.uuid have any security implications? Can you describe it?

@tianshimoyi Sure.
From the results of Docker scan, we can see the following results #18 :

✗ High severity vulnerability found in github.com/satori/go.uuid
  Description: Insecure Randomness
  Info: https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMSATORIGOUUID-72488
  Introduced through: github.com/satori/[email protected]
  From: github.com/satori/[email protected]

Then I imported the image to the Snyk platform, and the detailed report is as follows:

Overview:
github.com/satori/go.uuid provides a pure Go implementation of Universally Unique Identifier (UUID).

Affected versions of this package are vulnerable to Insecure Randomness, which produces predictable UUID identifiers due to the limited number of bytes read when using the g.rand.Read function.

Note: This vulnerability was introduced on the master branch of satori/go.uuid after version 1.2.0. This was fixed in commit d91630c8510268e75203009fe7daf2b8e1d60c45.

I checked the commit history of the satori/go.uuid repository, and version 1.2.0 does not include this commit.

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

No branches or pull requests

2 participants