anodejs / node-farmjs Goto Github PK

Reported by @yosefd:

For example, when sending the request http://anodejs.cloudapp.net/pull.sys/history?$inst=X, if the request will be proxies, the actual URL the app will see if /pull.sys/history (instead of just /history).

It seems that it could be easily remedied with:

if (stream !== process.stdout) {
    stream.end();
}

From: Elad Ben-Israel
Sent: יום ד 11 ינואר 2012 13:01
To: Yosef Dinerstein; Saar Yahalom; Ami Turgman; Fadi Haik
Subject: Re: Uncauht exception from everywhere in anodejs

Let me disable capturing stdout from child processes in reverse proxy. It might be the cause.
I will need your help in recycling the farms because I don't have RDP access…
Elad.

From: Yosef Dinerstein [email protected]
Date: Wed, 11 Jan 2012 12:25:31 +0200
To: ELAD BEN-ISRAEL [email protected], Saar Yahalom [email protected], Ami Turgman [email protected], Fadi Haik [email protected]
Subject: Uncauht exception from everywhere in anodejs

This happens on many apps in huge volume:

{
    "level": "error",
    "message": " [\"Uncaught exception:\",\"Error: process.stdout cannot be closed\\n    at Socket.<anonymous> (node.js:284:15)\\n    at Object.afterWrite [as oncomplete] (net.js:478:10)\"]",
    "meta": {
      "time": "2012-01-11T08:06:06.384Z",
      "app": "anode.sys",
      "domain": "anodejs.cloudapp.net",
      "instanceId": "anodejsrole_IN_0",
      "requestID": null,
      "stackTop": "logException (c:\\an\\repo\\master\\node_modules\\anode\\shimme.js:68:48)"
    },
    "_id": "4f0d4331c13c68202401fa63"
  },

I guess, this also the trigger to huge logs size and hence the logging performance problem we found.

Aliases on branches should automatically resolved to apps on the same branch

Anode code is different from what is kept here.
Just fixed issue #379 of anode (should use full name and not base name header) on anode repository.

Need to

1. Update this repository with latest from anode (my fixes and multiple other changes).
2. Add test case to cover #379
3. Update version and publish the updated package.