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

Plus in protocol#1654

Closed
mmalecki wants to merge 2 commits intonodejs:masterfrom
mmalecki:plus-in-protocol
Closed

Plus in protocol#1654
mmalecki wants to merge 2 commits intonodejs:masterfrom
mmalecki:plus-in-protocol

Conversation

@mmalecki
Copy link
Copy Markdown

@mmalecki mmalecki commented Sep 5, 2011

As per @isaacs' comment in #1580, allowing plus character in protocol.

@bnoordhuis
Copy link
Copy Markdown
Member

Thanks, Maciej. Merged in d055294 and fb93ab4.

@bnoordhuis bnoordhuis closed this Sep 6, 2011
lyonplus pushed a commit to lyonplus/node that referenced this pull request May 22, 2015
PR-URL: nodejs/node#1679

Notable Changes:

* win,node-gyp: the delay-load hook for windows addons has now been
correctly enabled by default, it had wrongly defaulted to off in the
release version of 2.0.0 (Bert Belder) nodejs#1433
* os: tmpdir()'s trailing slash stripping has been refined to fix an
issue when the temp directory is at '/'. Also considers which slash is
used by the operating system. (cjihrig) nodejs#1673
* tls: default ciphers have been updated to use gcm and aes128 (Mike
MacCana) nodejs#1660
* build: v8 snapshots have been re-enabled by default as suggested by
the v8 team, since prior security issues have been resolved. This
should give some perf improvements to both startup and vm context
creation. (Trevor Norris) nodejs#1663
* src: fixed preload modules not working when other flags were used
before --require (Yosuke Furukawa) nodejs#1694
* dgram: fixed send()'s callback not being asynchronous (Yosuke
Furukawa) nodejs#1313
* readline: emitKeys now keeps buffering data until it has enough to
parse. This fixes an issue with parsing split escapes. (Alex Kocharin)
* cluster: works now properly emit 'disconnect' to cluser.worker (Oleg
Elifantiev) nodejs#1386
events: uncaught errors now provide some context (Evan Lucas) nodejs#1654
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants