Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update x/crypto and x/text dependencies #312

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

ajborley
Copy link
Contributor

Signed-off-by: Andrew Borley BORLEY@uk.ibm.com

@grdryn
Copy link
Contributor

grdryn commented Jul 5, 2022

@ajborley Hi! Without this fix, when I scan with snyk, I didn't seem to get anything about these. Just curious, did you scan with something else, or use special parameters or something?

@ajborley
Copy link
Contributor Author

ajborley commented Aug 3, 2022

@ajborley Hi! Without this fix, when I scan with snyk, I didn't seem to get anything about these. Just curious, did you scan with something else, or use special parameters or something?

Hi @grdryn - sorry for the delay. This was spotted by our corporate scanning tool, although I'm not sure exactly how it is configured. Does snyk still not report anything? I would expect CVE-2021-38561 to show up at least, given that it's in the snyk db.

@danielqsj
Copy link
Owner

LGTM, thanks @ajborley

@danielqsj danielqsj merged commit d6050b1 into danielqsj:master Aug 23, 2022
@ajborley ajborley deleted the update-crypto-and-text-deps branch August 23, 2022 07:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants