Skip to content

Support rest #9191

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
gangxie112 opened this issue May 22, 2022 · 3 comments
Open

Support rest #9191

gangxie112 opened this issue May 22, 2022 · 3 comments

Comments

@gangxie112
Copy link

Is your feature request related to a problem?

Rest is still play an import part in real projects. If the grpc framework could support it, it will be great. It seems that grpc donet still have some experimental project for it : grpc/grpc-dotnet#167

Describe the solution you'd like

Describe alternatives you've considered

Additional context

@ejona86
Copy link
Member

ejona86 commented Jun 13, 2022

This is a very large request and would need substantial investment to support, at least in C and Java (Go might be a bit easier, but it would still be involved). I'm happy it has been added in .Net, but that doesn't change much. For this we'd need a team to step up and be prepared to do the work and maintenance. And there's a lot of work involved to make the core support plain HTTP.

@ejona86 ejona86 added this to the Unscheduled milestone Jun 13, 2022
@Huddie
Copy link

Huddie commented Aug 17, 2022

@ejona86 what's the best workaround atm in java to support this?

@ST-DDT
Copy link
Contributor

ST-DDT commented Oct 1, 2022

This one worked at some point, but it might need a little touch up.
https://github.com/yidongnan/grpc-spring-boot-starter/pull/207
I wouldnt call it ready for production yet.
I plan to improve it in the future if I have time for it. Please keep in mind that it uses a secondary webserver to serve the http requests/endpoint.

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

No branches or pull requests

4 participants