cmullaparthi / ibrowse

Erlang HTTP client
Other
515 stars 191 forks source link

sock options ignored on ssl #145

Closed sdancer closed 7 years ago

sdancer commented 8 years ago

last commit is, i dont need autorship actually, u can apply the patch :P

cmullaparthi commented 8 years ago

Thank you @shakugan. The patch looks good but it needs a tweak. The ssl:connect should be invoked with the timeout value in State#state.connect_timeout. If you can add this and resubmit the pull request I will accept it.