Functional HTTP Client!

deanberris · deanberris · commit 833469530491 · 2012-03-16T22:36:57.000+11:00
So, now this particular non-HTTPS implementation is doing what it's
supposed to be doing. There's still a bug in the HTTP implementation
which needs some squashing, but I'm proud to announce that we're now in
(almost) feature parity with the 0.9.3 implementation, *without the
header-only dogma*.

There's still quite a lot to do, but this is a milestone commit for the
refactoring effort.
diff --git a/boost/network/protocol/http/client/connection/async_normal.ipp b/boost/network/protocol/http/client/connection/async_normal.ipp
@@ -99,6 +99,7 @@ struct http_async_connection_pimpl : boost::enable_shared_from_this<http_async_c
     accessor.set_headers_promise(r, this->headers_promise);
     accessor.set_body_promise(r, this->body_promise);
     accessor.set_version_promise(r, this->version_promise);
+    accessor.set_status_promise(r, this->status_promise);
     accessor.set_status_message_promise(r, this->status_message_promise);
     BOOST_NETWORK_MESSAGE("futures and promises lined up.");
   }
diff --git a/libs/network/test/http/client_get_test.cpp b/libs/network/test/http/client_get_test.cpp
@@ -25,8 +25,8 @@ BOOST_AUTO_TEST_CASE(http_client_get_test) {
     response.get_status(status_);
     response.get_status_message(status_message_);
     BOOST_CHECK_EQUAL ( version_.substr(0,7), "HTTP/1.");
-    BOOST_CHECK_EQUAL ( status_, 200u );
-    BOOST_CHECK_EQUAL ( status_message_, std::string("OK") );
+    BOOST_CHECK_EQUAL ( status_, 302u );
+    BOOST_CHECK_EQUAL ( status_message_, std::string("Found") );
 }
 
 #ifdef BOOST_NETWORK_ENABLE_HTTPS

Original file line number	Diff line number	Diff line change
`@@ -99,6 +99,7 @@ struct http_async_connection_pimpl : boost::enable_shared_from_this<http_async_c`
`99`	`99`	`accessor.set_headers_promise(r, this->headers_promise);`
`100`	`100`	`accessor.set_body_promise(r, this->body_promise);`
`101`	`101`	`accessor.set_version_promise(r, this->version_promise);`
	`102`	`+ accessor.set_status_promise(r, this->status_promise);`
`102`	`103`	`accessor.set_status_message_promise(r, this->status_message_promise);`
`103`	`104`	`BOOST_NETWORK_MESSAGE("futures and promises lined up.");`
`104`	`105`	`}`