Skip to content

Refactored HTTP client v2. #476

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Jan 2, 2015
Merged

Refactored HTTP client v2. #476

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
0b49d2b
Added a facility to track upload/download progress.
glynos Feb 16, 2014
1bc435b
Removed ssl_connection from client.
glynos Feb 17, 2014
2b1c0e2
Added transfer direction callback.
glynos Mar 23, 2014
d00f99f
Renamed request_helper as request_context.
glynos Mar 23, 2014
47ced49
Updated uri submodule.
glynos Mar 26, 2014
03dddb2
Removed noexcept from destructors; added some flags for chunked trans…
glynos Apr 12, 2014
ccebaeb
Formatted source code and add source comments.
glynos May 11, 2014
ad273ec
Added some comments; fixed tests so that they compile for Boost.Optio…
glynos Sep 10, 2014
862263d
Updated uri submodule.
glynos Sep 10, 2014
4f2f0e2
Formatted files with clang-format.
glynos Sep 10, 2014
907a176
Updated uri submodule.
glynos Sep 21, 2014
ff5f714
Merge remote-tracking branch 'upstream/master'
glynos Dec 19, 2014
4e0add7
Use std::unique_ptr for pimpl idiom in HTTP client.
glynos Dec 24, 2014
6771a55
Added connection timeout check to HTTP client.
glynos Dec 24, 2014
50f3a4c
Fixed bug in error handling in the HTTP client.
glynos Dec 24, 2014
e2da305
Minor refactoring in the HTTP client.
glynos Dec 24, 2014
42ab79d
Simplified the HTTP client, request and response classes.
glynos Dec 25, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed ssl_connection from client.
  • Loading branch information
@glynos
glynos committed Sep 20, 2014
commit 1bc435b5c7d3b77f62675d71cd8324525656a3fa
8 changes: 1 addition & 7 deletions http/src/http/v2/client/client.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -346,13 +346,7 @@ std::future<response> client::execute(request req, request_options options) {
}
else {
// TODO factory based on HTTP or HTTPS
if (req.is_https()) {
connection = std::make_shared<client_connection::ssl_connection>(pimpl_->io_service_,
pimpl_->options_);
}
else {
connection = std::make_shared<client_connection::normal_connection>(pimpl_->io_service_);
}
connection = std::make_shared<client_connection::normal_connection>(pimpl_->io_service_);
}
return pimpl_->execute(std::make_shared<request_helper>(connection, req, options));
}
Expand Down