Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

simplify tls connection setup and make compatible with node v0.8 #438

Closed
wants to merge 2 commits into from

2 participants

@natevw

(note that, as discussed on #437, this makes no attempt at compatibility with node <= v0.4, because AFAICT the .forever() agent was not working there already.)

@mikeal
Owner

please remove the version change. i keep master .odd until I do a release of .even and check the package.json change in by itself.

@mikeal
Owner

this doesn't merge cleanly, please reopen when it does.

@mikeal mikeal closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 7 deletions.
  1. +1 −7 forever.js
View
8 forever.js
@@ -93,11 +93,5 @@ function ForeverAgentSSL (options) {
}
util.inherits(ForeverAgentSSL, ForeverAgent)
-ForeverAgentSSL.prototype.createConnection = createConnectionSSL
+ForeverAgentSSL.prototype.createConnection = tls.connect
ForeverAgentSSL.prototype.addRequestNoreuse = AgentSSL.prototype.addRequest
-
-function createConnectionSSL (port, host, options) {
- options.port = port
- options.host = host
- return tls.connect(options)
-}
Something went wrong with that request. Please try again.