[Merge] lp:~pete-woods/net-cpp/uri-builder into lp:net-cpp
Pete Woods
pete.woods at canonical.com
Wed Jun 11 08:41:02 UTC 2014
Hmm, just started looking at this, and the httpbin paths aren't really in a useful form for this.
i.e. they provide:
base = "http://127.0.0.1:5000"
path = "/get/banana"
To put these in a useful form for the API, I'd first have to split them into their components, then pass those into the URI builder. Seems like a lot of effort to go through.
If I just ran them through like this:
net::make_uri(httpbin::host, {httpbin::resources::get()})
then it will URI escape the slash characters, which is not what we want at all.
The alternative would be to do this:
net::make_uri(httpbin::host + httpbin::resources::get())
but I'm not sure if that's any better than just using a string?
--
https://code.launchpad.net/~pete-woods/net-cpp/uri-builder/+merge/222614
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~pete-woods/net-cpp/uri-builder into lp:net-cpp.
More information about the Ubuntu-reviews
mailing list