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

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node-v0.x-archive
base: c7b84a1d01fa
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 862b16420918
Choose a head ref
  • 2 commits
  • 6 files changed
  • 2 contributors

Commits on Feb 11, 2013

  1. net: don't suppress ECONNRESET 

    Let ECONNRESET network errors bubble up so clients can detect them.

Commit c4454d2 suppressed and turned them into regular end-of-stream
events to fix the then-failing simple/test-regress-GH-1531 test. See
also issue #1571 for (scant) details.

It turns out that special handling is no longer necessary. Remove the
special casing and let the error bubble up naturally.

pummel/test-https-ci-reneg-attack and pummel/test-tls-ci-reneg-attack
are updated because they expected an EPIPE error code that is now an
ECONNRESET. Suppression of the ECONNRESET prevented the test from
detecting that the connection has been severed whereupon the next
write would fail with an EPIPE.

Fixes #1776.
    @bnoordhuis @isaacs
    bnoordhuis authored and isaacs committed Feb 11, 2013
    Copy the full SHA
    14a4245 View commit details
    Browse the repository at this point in the history

  2. test: update http tests for ECONNRESET change

    @isaacs
    isaacs committed Feb 11, 2013
    Copy the full SHA
    862b164 View commit details
    Browse the repository at this point in the history