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

Bump github.com/gorilla/websocket from 1.5.0 to 1.5.1 #2898

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 7, 2024

Bumps github.com/gorilla/websocket from 1.5.0 to 1.5.1.

Release notes

Sourced from github.com/gorilla/websocket's releases.

Release v1.5.1

What's Changed

New Contributors

Full Changelog: gorilla/websocket@v1.5.0...v1.5.1

Commits
  • ac0789b update GitHub workflows (#857)
  • 78c3487 update golang.org/x/net (#856)
  • 666c197 Update go version & add verification/testing tools (#840)
  • 8039329 Correct way to save memory using write buffer pool and freeing net.http defau...
  • 8983b96 Merge pull request #839 from gorilla/coreydaley-patch-1
  • 931041c Update README.md
  • 76ecc29 archive mode
  • af47554 check error before GotConn for trace
  • bc7ce89 Check for and report bad protocol in TLSClientConfig.NextProtos (#788)
  • 27d91a9 drop the versions of go that are no longer supported + add 1.18 to ci
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Feb 7, 2024
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/gorilla/websocket-1.5.1 branch 3 times, most recently from c863be3 to 59c2889 Compare February 7, 2024 15:53
@StevenACoffman StevenACoffman added help wanted Extra attention is needed websocket labels Feb 7, 2024
@StevenACoffman
Copy link
Collaborator

@telemenar @UnAfraid @vlad-tokarev @szgupta @zhixinwen Any of you websocket gqlgen contributors know why this upgrade is failing? I would appreciate someone taking a look! Thanks.

@UnAfraid
Copy link
Contributor

UnAfraid commented Feb 7, 2024

@StevenACoffman I'll have a look at it

Edit: gorilla/websocket#877
Edit 2: https://github.com/gorilla/websocket/pull/840/files#diff-4f427d2b022907c552328e63f137561f6de92396d7a6e8f6c2ea1bcf0db52654L190-R197 this is the culprit, we are no longer getting net.ErrClosed, because all the errors are being hidden away...
Edit 3: gorilla/websocket#894 i've opened pull to fix the issue upstream

@StevenACoffman
Copy link
Collaborator

Thanks so much!

@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/gorilla/websocket-1.5.1 branch 3 times, most recently from eb543df to 1d6ecbd Compare February 12, 2024 18:57
@StevenACoffman
Copy link
Collaborator

StevenACoffman commented Feb 19, 2024

So gorilla merged @UnAfraid PR (thanks!), but I see a lot of people have had a lot of other problems with gorilla 1.5.1 release. Since I don't use websockets, I'm not clear if it is wise to update gqlgen's gorilla/websocket dependency to the current mainline commit or if we should wait until the 1.5.2 release. People can try it out with:

go mod edit -replace github.com/gorilla/websocket=github.com/gorilla/websocket@695e9095ce8736ac99c83939ca6b0fe93768f680

If the current 1.5.0 is working for folks, I'm ok with waiting until 1.5.2, but if 1.5.0 is breaking someone's production, I'm also ok with going forward.

I'm going to let the decision rest with gqlgen's Websocket Council of Wisdom™️ @telemenar @UnAfraid @vlad-tokarev @szgupta @zhixinwen

@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/gorilla/websocket-1.5.1 branch from 1d6ecbd to 1610a25 Compare February 26, 2024 14:07
Bumps [github.com/gorilla/websocket](https://github.com/gorilla/websocket) from 1.5.0 to 1.5.1.
- [Release notes](https://github.com/gorilla/websocket/releases)
- [Commits](gorilla/websocket@v1.5.0...v1.5.1)

---
updated-dependencies:
- dependency-name: github.com/gorilla/websocket
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/gorilla/websocket-1.5.1 branch from 1610a25 to 2d825cd Compare March 4, 2024 13:51
@StevenACoffman
Copy link
Collaborator

For anyone watching, the 1.5.1 release had a number of problems, which gqlgen contributors have been making patches on. We're going to ignore 1.5.1 and hope that 1.5.2 is in better shape.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file go Pull requests that update Go code help wanted Extra attention is needed websocket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants