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: eadb4f56063f
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 28dbc96a6cd4
Choose a head ref
  • 3 commits
  • 4 files changed
  • 2 contributors

Commits on Feb 9, 2014

  1. test: fix test-http-pipeline-flood 

    The number of connections achieved by the test can vary by platform
and by machine. Lowering the acceptance threshold so that the
test passes on Windows.
    @orangemocha @indutny
    orangemocha authored and indutny committed Feb 9, 2014
    Copy the full SHA
    e3ec2f7 View commit details
    Browse the repository at this point in the history

  2. dns: verify argument is valid function in resolve 

    Don't use argument as callback if it's not a valid callback function.
Throw a valid exception instead explaining the issue.

Adds to #7070 ("DNS — Throw meaningful error(s)").
    @19h @indutny
    19h authored and indutny committed Feb 9, 2014
    Copy the full SHA
    abe4c34 View commit details
    Browse the repository at this point in the history

  3. dns: validate arguments in resolver 

    Mitigate C++-land assertion error, add test accordingly.

Fix #7070
    @19h @indutny
    19h authored and indutny committed Feb 9, 2014
    Copy the full SHA
    28dbc96 View commit details
    Browse the repository at this point in the history