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

wspb.go uses deprecated protobuf package #311

Closed
iamcalledrob opened this issue Aug 2, 2021 · 4 comments
Closed

wspb.go uses deprecated protobuf package #311

iamcalledrob opened this issue Aug 2, 2021 · 4 comments
Milestone

Comments

@iamcalledrob
Copy link

I love the idea of the protobuf helpers in wspb.go, but when trying to use them I noticed that wspb.go imports github.com/golang/protobuf/proto. Since this module is deprecated, this helper can't easily be used in a codebase using the newer protobuf library (google.golang.org/protobuf).

From https://github.com/golang/protobuf:

It has been superseded by the google.golang.org/protobuf module, which contains an updated and simplified API, support for protobuf reflection, and many other improvements. We recommend that new code use the google.golang.org/protobuf module.

It would be wonderful if this import could be switched out to use the newer module 😀

@nhooyr
Copy link
Owner

nhooyr commented Aug 6, 2021

Oh very nice! Will definitely do in the next major release as it would be a breaking change.

Though perhaps this also suggests protobuf helpers don't belong in the core package and should be published elsewhere?

@ash2k
Copy link

ash2k commented Jun 13, 2023

Yes, moving the helpers out e.g. into an example that can be copied or into a separate module would be good to avoid transitive dependencies.

@nhooyr nhooyr added this to the v1.8.8 milestone Sep 28, 2023
nhooyr added a commit that referenced this issue Oct 13, 2023
The library we're currently using for protobufs is deprecated. Doesn't belong in the library core
anyway.

Closes #311
nhooyr added a commit that referenced this issue Oct 13, 2023
The library we're currently using for protobufs is deprecated. Doesn't belong in the library core
anyway.

Closes #311
Updates #297
@nhooyr
Copy link
Owner

nhooyr commented Oct 13, 2023

Done in dev.

@nhooyr nhooyr closed this as completed Oct 13, 2023
nhooyr added a commit that referenced this issue Oct 13, 2023
The library we're currently using for protobufs is deprecated. Doesn't belong in the library core
anyway.

Closes #311
Updates #297
@xornet-sl
Copy link

Is this a joke? Why wspb - a public package got removed?!
Guys, you exposed this, people use it, it is NOT an internal package. Why you just brake everything in PATCH release??!!
I even can not re-create the functionality of wspb as it uses internal package. Why you did that? Was it really a problem just to update the needed dependencies and let it live?

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

No branches or pull requests

4 participants