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

Fixes 9.8 CRITICAL CVE-2020-26892 #1094

Closed
wants to merge 2 commits into from

Conversation

hashier
Copy link
Contributor

@hashier hashier commented May 29, 2021

Updating github.com/nats-io/nats-server/v2 to version 2.2.6...
Updating github.com/nats-io/nats.go to version 1.11.0...
@peterbourgon
Copy link
Member

@hashier Is that a true test failure? If so, can you please fix it?

@hashier
Copy link
Contributor Author

hashier commented May 29, 2021

I have unfortunately no idea about this library. It is just pulled in through a transitive dependency and I thought I let you all know.

@peterbourgon
Copy link
Member

Can't merge a PR with broken tests 🤷

@peterbourgon peterbourgon mentioned this pull request May 30, 2021
@peterbourgon
Copy link
Member

#1095

@hashier
Copy link
Contributor Author

hashier commented May 30, 2021

Thanks! Looking at your PR this would have taken me long since I have never used the lib and I probably would just have introduced another mistake.

Thanks again for taking it up.

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

Successfully merging this pull request may close these issues.

None yet

2 participants