Skip to content

CBG-1901: Update ISGR/Blip to nhooyr.io/websocket #5421

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

Merged
merged 4 commits into from
Feb 15, 2022
Merged

Conversation

bbrks
Copy link
Member

@bbrks bbrks commented Feb 3, 2022

CBG-1901

Replaces go-blip and ISGR x/net/websocket implementations with nhooyr.io/websocket

TODO

  • More testing of mixed websocket client/server implementations.

Dependencies (if applicable)

Integration Tests

@bbrks bbrks requested a review from adamcfraser February 3, 2022 16:43
@bbrks bbrks changed the title Update ISGR/Blip to nhooyr.io/websocket CBG-1901: Update ISGR/Blip to nhooyr.io/websocket Feb 3, 2022
adamcfraser
adamcfraser previously approved these changes Feb 4, 2022
Copy link
Collaborator

@adamcfraser adamcfraser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bbrks bbrks force-pushed the nhooyr-websocket branch 2 times, most recently from 59c56b7 to 236edd2 Compare February 4, 2022 09:47
@bbrks bbrks assigned bbrks and unassigned adamcfraser Feb 4, 2022
@bbrks bbrks assigned adamcfraser and unassigned bbrks Feb 9, 2022
@bbrks bbrks merged commit 9c79d65 into master Feb 15, 2022
@bbrks bbrks deleted the nhooyr-websocket branch February 15, 2022 11:35
gregns1 pushed a commit that referenced this pull request Jan 26, 2023
* Update ISGR/Blip to nhooyr.io/websocket

* Fix manifest path

* Update manifest for couchbasedeps compress

* Bump go-blip manifest
bbrks added a commit that referenced this pull request Jan 26, 2023
…6040)

* CBG-2611: Update go-blip version, and add websocket library and compress library

* Backport jenkinsfile updates to 2.8.3 (#5585)

* Backport jenkinsfile fixes to 2.8.3

* Remove 3.0-specific config

* fix manifest

* updated errCheck version

* updated go version version

* updated go version version again

* turn off go mod

* updates to fix failiure on jenkins

* CBG-2611: [2.8.4] Update ISGR/Blip to nhooyr.io/websocket (#5421)

* Update ISGR/Blip to nhooyr.io/websocket

* Fix manifest path

* Update manifest for couchbasedeps compress

* Bump go-blip manifest

* fixed errors woth dependencies and cherry picked CBG-2607 as it was needed to allow the dependency changes

* duplicate paths

* fix for some test

* test fixes + lite core skip

* fix go vet

Co-authored-by: Adam Fraser <[email protected]>
Co-authored-by: Ben Brooks <[email protected]>
gregns1 pushed a commit that referenced this pull request Jan 30, 2023
* Update ISGR/Blip to nhooyr.io/websocket

* Fix manifest path

* Update manifest for couchbasedeps compress

* Bump go-blip manifest
bbrks added a commit that referenced this pull request Jan 31, 2023
* CBG-2608: update websocket implementation

* CBG-1901: Update ISGR/Blip to nhooyr.io/websocket (#5421)

* Update ISGR/Blip to nhooyr.io/websocket

* Fix manifest path

* Update manifest for couchbasedeps compress

* Bump go-blip manifest

* CBG-2242: fix errors.As in tests (#5739)

* errors as fix

* Assert on typed x509 error first before macOS-specific fallback (#6011)

Only falls back to macOS-specific x509 error assertion if the initial
typed assertion failed.

Fixes test failure when running with Go 1.20, 1.19.5 or 1.18.10
due to golang/go#56891

* missed dependency upgrades to json iterator, and reflect to stop panics

---------

Co-authored-by: Ben Brooks <[email protected]>
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.

2 participants