Skip to content

Conversation

jvonniedapn
Copy link
Contributor

Fixes a bug that causes HTTPS to never be used. Without using the HttpsAgent version of KeepAliveAgent the connections always default to http instead of https, even when https is specified as the protocol.

…pAgent version of KeepAliveAgent the connections always default to http instead of https, even when https is specified as the protocol.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e964863 on PlayNetwork:master into 58cc633 on elasticsearch:master.

@spalger
Copy link
Contributor

spalger commented Jan 14, 2014

Wow, good catch. Thanks for the patch, have you completed step 5 in CONTRIBUTING.md?

@jvonniedapn
Copy link
Contributor Author

Yea, just did.

spalger pushed a commit that referenced this pull request Jan 14, 2014
Fixes a bug that causes HTTPS to never be used.
@spalger spalger merged commit d7c278f into elastic:master Jan 14, 2014
spalger pushed a commit that referenced this pull request Jan 14, 2014
@spalger
Copy link
Contributor

spalger commented Jan 14, 2014

Thanks @jvonniedapn!

cdituri pushed a commit to cdituri/elasticsearch-js that referenced this pull request Jun 14, 2017
…03064944

Exec tachikoma update 20150803064944
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants