Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tls: proxy handle.reading back to parent handle #1004

Closed
wants to merge 1 commit into from

Conversation

indutny
Copy link
Member

@indutny indutny commented Feb 28, 2015

Fix: #995

cc @rvagg this should be a faster version of #1003

Please verify it and if it works - land it instead of #1003

@rvagg
Copy link
Member

rvagg commented Feb 28, 2015

indutny added a commit that referenced this pull request Feb 28, 2015
Fix: #995
PR-URL: #1004
Reviewed-By: Rod Vagg <rod@vagg.org>
@rvagg
Copy link
Member

rvagg commented Feb 28, 2015

landed @ 1ab7e80

@rvagg rvagg closed this Feb 28, 2015
@rvagg rvagg mentioned this pull request Feb 28, 2015
rvagg added a commit that referenced this pull request Feb 28, 2015
Notable changes:

* tls: A typo introduced in the TLSWrap changes in
  #840 only encountered as a bug on
  Windows was not caught by the io.js CI system due to problems with the
  Windows build script and the Windows CI slave configuration, see
  Fixed in #994 &
  #1004 (Fedor Indutny)
* npm: Upgrade npm to 2.6.1. See
  https://github.com/npm/npm/blob/master/CHANGELOG.md#v260-2015-02-12
  for details.
* Add new collaborators:
  - Robert Kowalski (@robertkowalski)
  - Christian Vaagland Tellnes (@tellnes)
  - Brian White (@mscdex)
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.

2 participants