thrift icon indicating copy to clipboard operation
thrift copied to clipboard

Published 20 hours ago • verified publisher icon maximg

Implement HTTP transport

Open maximg opened this issue 10 years ago • 4 comments

Required for acceptance in Thrift mainline

maximg avatar Jan 28 '15 15:01 maximg

This doesn't seem too difficult, I'm going to take a shot at this.

gsingh93 avatar May 31 '15 19:05 gsingh93

I have some basic stuff working, but after talking to some people on IRC/the mailing list, I don't think this is a priority.

I think the first step should be to do the bare minimum to get this merged into the mainline, and from what I understand that is essentially getting a working tutorial client/server implementation. So I'll be looking into that instead (and I'll post a link to my work in progress here a little later just for reference or if anyone wants to continue).

gsingh93 avatar Jun 01 '15 21:06 gsingh93

Here is my work in progress branch: https://github.com/gsingh93/thrift/tree/http-transport

It was actually almost complete, but I couldn't properly test it with other servers. I may return to it later, or someone might be able to finish it off first.

gsingh93 avatar Jun 01 '15 23:06 gsingh93

Pasting here the link to the discussion. Some good info there, re multiplex transport as well.

So the conclusion is we will deprioritize HTTP for the drop to main line.

maximg avatar Jun 06 '15 18:06 maximg