> On Jan. 15, 2014, 2:49 a.m., Benjamin Hindman wrote: > > 3rdparty/libprocess/src/process.cpp, line 3667 > > <https://reviews.apache.org/r/16839/diff/5/?file=423523#file423523line3667> > > > > I'd love to see a TODO which let's us specify the content type as a 6th > > parameter. In the mean time, can you leave a comment as to why you picked > > application/x-ww-form-urlencoded as the default?
great point re: content type. I'm going to go ahead and add the argument. the signature is going to now be: Future<Response> post(const UPID& upid, const string& path, const string& contentType, const string& body) path can't have an optional since contentType & body should be required for a post. I thought about making it the last argument but it's easy enough for the caller to pass in an explicit empty string and have no path seems pretty uncommon anyway. - Charlie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16839/#review31846 ----------------------------------------------------------- On Jan. 15, 2014, 2:05 a.m., Charlie Carson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16839/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2014, 2:05 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Jeff Currier, and Jie > Yu. > > > Bugs: https://issues.apache.org/jira/browse/MESOS-902 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/MESOS-902 > > > Repository: mesos-git > > > Description > ------- > > Add post function to http.hpp of libprocess. > > This adds the post equivalent of the get function to http.hpp. > > The existing get method is refactored into internal::httpRequest > with arguments for method, query, and body. > > The get and post functions can then be implemented as simple > wrapper of internal::httpRequest. > > There are also new unit tests to verify the existing behavior of > get and the new post behavior. > > See: https://issues.apache.org/jira/browse/MESOS-902 > > Review: https://reviews.apache.org/r/16839 > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp > 5bdd520c9e0bcc0a2d3a4554cc4ced99dcf78b51 > 3rdparty/libprocess/src/process.cpp > 67f7f9b3b05c8bc9b0f6281689223996ddfa68d1 > 3rdparty/libprocess/src/tests/http_tests.cpp > 68d1a1b2ed5645d861b1e613aed9731368ebdc6a > > Diff: https://reviews.apache.org/r/16839/diff/ > > > Testing > ------- > > added new unit tests > make check > > > Thanks, > > Charlie Carson > >
