Create a gist now

Instantly share code, notes, and snippets.

What would you like to do?
var http = require("http");
var sys = require("sys");
function sendReq()
var resent = false;
function resendReq(reason)
sys.print(reason + "\n");
resent = true;
var client = http.createClient(80, "");
client.on("error", function(){ resendReq("client error") });
client.on("close", function(){ resendReq("client close") });
var request = client.request("GET", "/", { Host: "" });
request.on("error", function(){ resendReq("request error") });
request.on("close", function(){ resendReq("request close") });
var body = "";
request.on("response", function(response){
response.on("error", function(){ resendReq("response error") });
response.on("close", function(){ resendReq("response close") });
response.on("data", function(chunk){
body = body + chunk;
response.on("end", function(){ resendReq("response end") });

r14c commented Dec 6, 2010

I'm not sure this constitutes a memory leak because you are creating a lot of http.createClient instances and attaching events listeners to them that never expire. If you moved the client object out of the sendReq function the "leak" should disappear.


richcollins commented Dec 6, 2010

this updated version should work. sendReq was being called twice, once for end and once for close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment