mirror of https://github.com/lukechilds/node.git
Browse Source
- fix crash calling ClientRequest::setKeepAlive if the underlying request is HTTPS. - fix discarding of callback parameter when calling ClientRequest::setTimeout on HTTPS requests. - fix discarding of noDelay parameter when calling ClientRequest::setNoDelay on HTTPS requests.v0.9.1-release
Jimb Esser
13 years ago
committed by
Ben Noordhuis
2 changed files with 92 additions and 4 deletions
@ -0,0 +1,83 @@ |
|||
// Copyright Joyent, Inc. and other Node contributors.
|
|||
//
|
|||
// Permission is hereby granted, free of charge, to any person obtaining a
|
|||
// copy of this software and associated documentation files (the
|
|||
// "Software"), to deal in the Software without restriction, including
|
|||
// without limitation the rights to use, copy, modify, merge, publish,
|
|||
// distribute, sublicense, and/or sell copies of the Software, and to permit
|
|||
// persons to whom the Software is furnished to do so, subject to the
|
|||
// following conditions:
|
|||
//
|
|||
// The above copyright notice and this permission notice shall be included
|
|||
// in all copies or substantial portions of the Software.
|
|||
//
|
|||
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
|
|||
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
|||
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
|
|||
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
|
|||
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
|
|||
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
|
|||
// USE OR OTHER DEALINGS IN THE SOFTWARE.
|
|||
|
|||
|
|||
|
|||
|
|||
if (!process.versions.openssl) { |
|||
console.error('Skipping because node compiled without OpenSSL.'); |
|||
process.exit(0); |
|||
} |
|||
|
|||
var common = require('../common'); |
|||
var assert = require('assert'); |
|||
|
|||
var fs = require('fs'); |
|||
var exec = require('child_process').exec; |
|||
|
|||
var http = require('http'); |
|||
var https = require('https'); |
|||
|
|||
var options = { |
|||
key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'), |
|||
cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem') |
|||
}; |
|||
|
|||
var body = 'hello world\n'; |
|||
|
|||
// Try first with http server
|
|||
|
|||
var server_http = http.createServer(function(req, res) { |
|||
console.log('got HTTP request'); |
|||
res.writeHead(200, { 'content-type': 'text/plain' }); |
|||
res.end(body); |
|||
}); |
|||
|
|||
|
|||
server_http.listen(common.PORT, function() { |
|||
var req = http.request({ port: common.PORT }, function(res) { |
|||
server_http.close(); |
|||
}); |
|||
// These methods should exist on the request and get passed down to the socket
|
|||
req.setNoDelay(true); |
|||
req.setTimeout(1000, function() { }); |
|||
req.setSocketKeepAlive(true, 1000); |
|||
req.end(); |
|||
}); |
|||
|
|||
// Then try https server (requires functions to be mirroed in tls.js's CryptoStream)
|
|||
|
|||
var server_https = https.createServer(options, function(req, res) { |
|||
console.log('got HTTPS request'); |
|||
res.writeHead(200, { 'content-type': 'text/plain' }); |
|||
res.end(body); |
|||
}); |
|||
|
|||
server_https.listen(common.PORT+1, function() { |
|||
var req = https.request({ port: common.PORT+1 }, function(res) { |
|||
server_https.close(); |
|||
}); |
|||
// These methods should exist on the request and get passed down to the socket
|
|||
req.setNoDelay(true); |
|||
req.setTimeout(1000, function() { }); |
|||
req.setSocketKeepAlive(true, 1000); |
|||
req.end(); |
|||
}); |
Loading…
Reference in new issue