1. Apr 08, 2013
  2. Apr 06, 2013
  3. Apr 05, 2013
  4. Apr 04, 2013
  5. Apr 03, 2013
    • Brian White's avatar
      src: remove unused variables · 38106da7
      Brian White authored
      38106da7
    • isaacs's avatar
      http client: Ensure socket cleanup on response end · 234fb122
      isaacs authored
      If an http response has an 'end' handler that throws, then the socket
      will never be released back into the pool.
      
      Granted, we do NOT guarantee that throwing will never have adverse
      effects on Node internal state.  Such a guarantee cannot be reasonably
      made in a shared-global mutable-state side-effecty language like
      JavaScript.  However, in this case, it's a rather trivial patch to
      increase our resilience a little bit, so it seems like a win.
      
      There is no semantic change in this case, except that some event
      listeners are removed, and the `'free'` event is emitted on nextTick, so
      that you can schedule another request which will re-use the same socket.
      From the user's point of view, there should be no detectable difference.
      
      Closes #5107
      234fb122
  6. Apr 02, 2013
    • Andrew Hart's avatar
      test: test intended code-paths · db8ce89f
      Andrew Hart authored
      The tests did not agree with the test comments. Tests first and second
      were both testing the !state.reading case. Now second tests the
      state.reading && state.length case.
      
      Fixes joyent/node#5183
      db8ce89f
    • isaacs's avatar
      tls: Destroy socket when encrypted side closes · 164d5b34
      isaacs authored
      The v0.8 Stream.pipe() method automatically destroyed the destination
      stream whenever the src stream closed.  However, this caused a lot of
      problems, and was removed by popular demand.  (Many userland modules
      still have a no-op destroy() method just because of this.) It was also
      very hazardous because this would be done even if { end: false } was
      passed in the pipe options.
      
      In v0.10, we decided that the 'close' event and destroy() method are
      application-specific, and pipe() doesn't automatically call destroy().
      However, TLS actually depended (silently) on this behavior.  So, in this
      case, we should just go ahead and destroy the thing when close happens.
      
      Closes #5145
      164d5b34
    • Michael Hart's avatar
      440dcae9
  7. Apr 01, 2013
  8. Mar 31, 2013
  9. Mar 30, 2013
  10. Mar 29, 2013