Permalink
Switch branches/tags
Commits on Oct 9, 2014
  1. stream: remove duplicated expression

    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    Reviewed-by: Chris Dickinson <christopher.s.dickinson@gmail.com>
    yorkie committed with chrisdickinson Oct 2, 2014
Commits on Oct 1, 2014
  1. net: remove use of arguments in Server constructor

    The current implementation uses the arguments object in the Server()
    constructor. Since both arguments to Server() are optional, there was a
    high likelihood of accessing a non-existent element in arguments, which
    carries a performance overhead. This commit replaces the arguments
    object with named arguments.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    cjihrig committed with trevnorris Sep 24, 2014
Commits on Sep 17, 2014
  1. Merge remote-tracking branch 'upstream/v0.10' into v0.12

    Conflicts:
    	ChangeLog
    	deps/v8/src/hydrogen.cc
    	lib/http.js
    	lib/querystring.js
    	src/node_crypto.cc
    	src/node_version.h
    	test/simple/test-querystring.js
    tjfontaine committed Sep 17, 2014
Commits on Sep 16, 2014
  1. Now working on 0.10.33

    tjfontaine committed Sep 16, 2014
  2. 2014.09.16, Version 0.10.32 (Stable)

    * npm: Update to 1.4.28
    
    * v8: fix a crash introduced by previous release (Fedor Indutny)
    
    * configure: add --openssl-no-asm flag (Fedor Indutny)
    
    * crypto: use domains for any callback-taking method (Chris Dickinson)
    
    * http: do not send `0rnrn` in TE HEAD responses (Fedor Indutny)
    
    * querystring: fix unescape override (Tristan Berger)
    
    * url: Add support for RFC 3490 separators (Mathias Bynens)
    tjfontaine committed Sep 16, 2014
  3. npm: Update to 1.4.28

    tjfontaine committed Sep 16, 2014
  4. http: do not send `0\r\n\r\n` in TE HEAD responses

    When replying to a HEAD request, do not attempt to send the trailers and
    EOF sequence (`0\r\n\r\n`). The HEAD request MUST not have body.
    
    Quote from RFC:
    
    The presence of a message body in a response depends on both the
    request method to which it is responding and the response status code
    (Section 3.1.2).  Responses to the HEAD request method (Section 4.3.2
    of [RFC7231]) never include a message body because the associated
    response header fields (e.g., Transfer-Encoding, Content-Length,
    etc.), if present, indicate only what their values would have been if
    the request method had been GET (Section 4.3.1 of [RFC7231]).
    
    fix #8361
    
    Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
    indutny committed with tjfontaine Sep 13, 2014
  5. crypto: use domains for any callback-taking method

    This adds domains coverage for pdbkdf2, pseudoRandomBytes, and randomBytes.
    All others should be covered by event emitters.
    
    Fixes #5801.
    
    Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
    chrisdickinson committed with tjfontaine Sep 16, 2014
  6. lib, src: add vm.runInDebugContext()

    Compiles and executes source code in V8's debugger context.  Provides
    a programmatic way to get access to the debug object by executing:
    
        var Debug = vm.runInDebugContext('Debug');
    
    Fixes #7886.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  7. dns: include host name in error message if available

    This makes errors more readable and similar to FS errors, which also
    include file name.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    mmalecki committed with indutny Aug 6, 2014
Commits on Sep 15, 2014
  1. doc: document `process.env` better

    Fixes #6424.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    mmalecki committed with indutny Nov 11, 2013
  2. tls: fix encoding in certificate-related functions

    Strings are treated as UTF8 instead of one-byte strings when
    names are processed and when OpenSSL's ..._print functions are used.
    
    This commit fixes simple/test-tls-peer-certificate-encoding test.
    
    fix #8366
    alippai committed with indutny Sep 13, 2014
  3. doc: fix modules require.resolve documentation

    The behavior of the `node_modules` lookup algorithm was
    changed in #1177, but the documentation was not updated completely
    to describe the new behavior.
    
    The pseudocode of the lookup algorithm did not metion that
    `index.json` is tried to be loaded if you require a folder.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    lo1tuma committed with indutny Sep 3, 2014
Commits on Sep 9, 2014
  1. deps: backport 60c316 from v8 trunk

    Original commit message:
    
        Extend the interceptor setter ASSERT to support the JSGlobalProxy case.
    
        BUG=v8:3463
        LOG=n
        R=dcarney@chromium.org
    
        Review URL: https://codereview.chromium.org/415973004
    
        git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22589 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
    
        Signed-off-by: Fedor Indutny <fedor@indutny.com>
    
    fix #7969
    indutny committed with trevnorris Jul 24, 2014
  2. gyp: fix post-mortem in v0.11

    Expose missing constants and keep symbols on OSX.
    indutny committed with trevnorris Jul 21, 2014
  3. deps: cherry-pick r21466 from v8 trunk

    Check for cached transition to ExternalArray elements kind.
    See [1] and [2] for details.
    
    [1] https://code.google.com/p/v8/issues/detail?id=3337
    [2] https://codereview.chromium.org/291193011
    
    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    bnoordhuis committed with trevnorris May 23, 2014
  4. deps: cherry-pick r21297 from v8 trunk

    Changes the return value of PropertyCallbackInfo<T>::This() from
    Local<Value> back to Local<Object>.  See [1] and [2] for background.
    
    [1] https://groups.google.com/forum/#!topic/v8-users/wP2UcQ4cBW4
    [2] https://codereview.chromium.org/285643008/
    
    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    bnoordhuis committed with trevnorris May 14, 2014
  5. deps: fix up v8 postmortem codegen

    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    bnoordhuis committed with trevnorris May 12, 2014
  6. v8: Upgrade 3.26.33 with 14 patches

    V8 3.26.31 has received 14 patches since the upgrade to 3.26.33. Since
    3.26.33 is technically a tag on the 3.27 branch, reverting back to
    3.26.31 would remove now default functionality like WeakMaps. Because of
    that the patches have simply been cherry-picked and squashed.
    
    Here is a summary of all patches:
    
    * Fix index register assignment in LoadFieldByIndex for arm, arm64, and
      mips.
    * Fix invalid attributes when generalizing because of incompatible map
      change.
    * Skip write barriers when updating the weak hash table.
    * MIPS: Avoid HeapObject check in HStoreNamedField.
    * Do GC if CodeRange fails to allocate a block.
    * Array.concat: properly go to dictionary mode when required.
    * Keep CodeRange::current_allocation_block_index_ in range.
    * Grow heap slower if GC freed many global handles.
    * Do not eliminate bounds checks for "<const> - x".
    * Add missing map check to optimized f.apply(...).
    * In GrowMode, force the value to the right representation to avoid
      deopts between storing the length and storing the value.
    * Reduce max executable size limit.
    * Fix invalid condition in check elimination effects.
    * Fix off-by-one error in Array.concat slow mode check.
    
    For more information see: https://github.com/v8/v8/commits/3.26
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    trevnorris committed Sep 9, 2014
  7. tls: `checkServerIdentity` option

    Allow overriding `checkServerIdentity` function, when connecting to a
    TLS server.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    tlivings committed with indutny Sep 5, 2014
Commits on Sep 5, 2014
  1. src: remove Environment::GetCurrentChecked()

    There is only one call site that uses it and that can do the checks
    itself.  Removes ~15 lines of code.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Sep 5, 2014
  2. lib: unexport http.parsers

    Unexport the http.parsers freelist.  It was originally exported by Ryan
    in commit 0003c70 but the commit log doesn't mention why and it's never
    been documented.  It's unclear if there are any users.
    
    The lifecycle of parser objects changed recently and it seems better to
    not let people shoot themselves in the foot so easily.
    
    If it turns out there are actually users, we can always re-export it
    again - probably under a slightly different name, to force people to
    update their code to the new way of things.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  3. lib: http: poison parser references after freeing

    Make it a little harder to slip in use-after-free bugs by nulling out
    references to the parser object after handing it off to freeParser().
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  4. src: update and expand comments in base-object.h

    It's safe to call BaseObject::object() from your destructor _unless_
    the handle is weak; then it's the weak callback that is calling your
    destructor and the object will have been released by the time the
    destructor runs.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  5. lib, src: don't make http parser handles weak

    Weak handles put strain on the garbage collector and the parser handle
    doesn't need to be weak in the first place.  This change should improve
    GC times on busy servers a little.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  6. src: add ClearWrap() to util.h

    Counterpart to Wrap(), clears the previously assigned internal field.
    Will be used in an upcoming commit.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Sep 4, 2014
  7. src: fix handle leak in Parser::Execute()

    Fix a resource leak where an intermediate Local<Context> handle in
    Environment::GetCurrent() got leaked into whatever HandleScope was
    further up the stack.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    bnoordhuis committed with trevnorris Aug 22, 2014
  8. lib: improved forEach object performance

    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    majimboo committed with trevnorris Aug 14, 2014
Commits on Sep 4, 2014
  1. test: listen on exclusive port in cluster workers

    Test that listening on exclusive ports with the cluster module works
    correctly.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    cjihrig committed with trevnorris Sep 4, 2014
Commits on Sep 3, 2014
  1. net,dgram: workers can listen on exclusive ports

    Allow cluster workers to listen on exclusive ports for TCP and UDP,
    instead of forcing all calls to go through the cluster master.
    
    Fixes: #3856
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    Reviewed-by: Fedor Indutny <fedor@indutny.com>
    cjihrig committed with trevnorris Aug 22, 2014
  2. src: be more intelligent about use of "arguments"

    Use 'use strict' when there are named arguments and the arguments object
    is passed to apply(). Also pass named arguments to call() when the named
    argument is modified by the function.
    
    Suggested in
    nodejs#8302 (comment)
    
    Confirmed in
    nodejs#8302 (comment)
    
    Signed-off-by: Trevor Norris <trev.norris@gmail.com>
    trevnorris committed Sep 3, 2014
  3. tls: support multiple keys/certs

    Required to serve website with both ECDSA/RSA certificates.
    indutny committed Sep 1, 2014
  4. tls_wrap: fix use after free

    Do not free TLSCallbacks from StreamWrap. TLSCallbacks is bound to a V8
    object and should be collected by V8's GC.
    indutny committed Sep 1, 2014