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

Proposal: JDK-only server implementation (repurposed OkHttp Server) #11545

Open
bsideup opened this issue Sep 20, 2024 · 1 comment
Open

Proposal: JDK-only server implementation (repurposed OkHttp Server) #11545

bsideup opened this issue Sep 20, 2024 · 1 comment

Comments

@bsideup
Copy link

bsideup commented Sep 20, 2024

Is your feature request related to a problem?

Currently, there is no "zerodep" transport that uses JDK types only

Describe the solution you'd like

Repurpose OkHttpServer that does not in fact depend on OkHttp (only Okio but even that dependency can be removed) as a generic JDK-only server implementation.

Describe alternatives you've considered

Existing servers work fine but sometimes cause version conflicts. Also, Netty remains the reference server, and, while being performant, its implementation is harder to follow due to the async nature and Netty's way of doing things.

Additional context

I am working on https://github.com/bsideup/grpc-bidi ("push" for gRPC, a.k.a. client-side channels) and it needs to expose itself as a server, with ClientCall being the transport. Currently, it uses the Netty server impl, but it is rather complex, plus requires a heavyweight dependency.

@ejona86
Copy link
Member

ejona86 commented Sep 20, 2024

What version conflict is there with grpc-okhttp? Sounds like you want something small, and that is what grpc-okhttp is intended for. You would need to make a much stronger case for a new transport implementation. (I don't really see any real argument here; I just see an assertion.)

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

2 participants