- Notifications
You must be signed in to change notification settings - Fork425
Verify hostname according to rfc2818#455
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes from1 commit
a59ed53ec4b8832514f4b918ea9a82651b6ceed0b5b1f305bFile filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -19,7 +19,7 @@ namespace impl { | ||
| struct normal_delegate : connection_delegate { | ||
| normal_delegate(asio::io_service &service); | ||
| virtual void connect(asio::ip::tcp::endpoint &endpoint, std::string host, | ||
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. You also need to change the declaration of | ||
| function<void(system::error_code const &)> handler); | ||
| virtual void write( | ||
| asio::streambuf &command_streambuf, | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -19,7 +19,8 @@ boost::network::http::impl::normal_delegate::normal_delegate( | ||
| : service_(service) {} | ||
| void boost::network::http::impl::normal_delegate::connect( | ||
| asio::ip::tcp::endpoint & endpoint, | ||
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Can you please not add a space between the ampersand (&) and the variable? Please keep the code consistent in style. | ||
| std::string host, | ||
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. It looks like you're adding too many spaces here (or a tab character). Please align it properly with the above parameter. | ||
| function<void(system::error_code const &)> handler) { | ||
| socket_.reset(new asio::ip::tcp::socket(service_)); | ||
| socket_->async_connect(endpoint, handler); | ||