Make Request API consistent with itself and others

Request::from_url was incorrectly creating requests with unencoded URLs.

diff --git a/source/http/request.cpp b/source/http/request.cpp
index 01948592ef9c8033119bf86bfbf1ff40c66307f7..f050e0440084d0a0b447875fec014d7ef4940dee 100644 (file)
--- a/source/http/request.cpp
+++ b/source/http/request.cpp
@@ -12,7 +12,10 @@ namespace Http {
 Request::Request(const string &m, const string &p):
        method(m),
        path(p)
-{ }
+{
+       if(path.find(' ')!=string::npos)
+               throw invalid_argument("Request::Request");
+}
 
 string Request::str() const
 {
@@ -45,7 +48,7 @@ Request Request::from_url(const string &str)
        if(url.scheme!="http")
                throw invalid_argument("Request::from_url");
 
-       string path = url.path;
+       string path = urlencode(url.path);
        if(path.empty())
                path = "/";
        if(!url.query.empty())