Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Add tls.start() #848

Closed
wants to merge 1 commit into from
Closed

Add tls.start() #848

wants to merge 1 commit into from

Conversation

weaver
Copy link

@weaver weaver commented Mar 29, 2011

Here's the pull request for tls.start(); see mailing list thread here. Thanks!

@ry
Copy link

ry commented Apr 11, 2011

Hey Ben,
Do you have time to put together a test for this? Just something simple in the style of test/simple/test-https-connecting-to-http.js ?

@weaver
Copy link
Author

weaver commented Apr 12, 2011

Sure thing, I'll write something up.

@pquerna
Copy link

pquerna commented Jul 4, 2011

@weaver did you have a chance to make a test? Could you update the pull request? Thanks!

@weaver
Copy link
Author

weaver commented Jul 5, 2011

This slipped my mind, apologies! I'll update the pull request today.

@jamesonjlee
Copy link

is there a chance this could get expanded to tls.createSocket(options) on which you can later call connect(port, host, callback)?

@koichik
Copy link

koichik commented Jan 21, 2012

Sorry for the delay, this has been fixed in v0.7.0 (#2489). You can pass socket: mySocket to tls.connect().
Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants