Changeset 527 for Dev/trunk/src/server/util/couch.js

Timestamp:

03/23/14 17:26:12 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• Dropped request module for a simple one we wrote ourselves using the native Node modules. Hopefully this one will not suffer from the problem that sometimes empty bodies are returned even when the statuscode and content-length of the request are ok & present.
• Handle exceptions better in HTTPResult chain, they were hoisted in unknown responses before. Should we not include them in default processing, because they are special?

File:

• Dev/trunk/src/server/util/couch.js (modified) (3 diffs)

Dev/trunk/src/server/util/couch.js

@@ -1 +1 @@
 var request = require('./request')
   , _ = require('underscore')
+  , HTTPResult = require('./http-result')
   ;
 
@@ -42 +43 @@
 }
 
-function couchRequest (method, url, body, opts) {
+function couchRequest (method, url, body, opts, retries) {
+    body = body || {};
+    retries = _.isNumber(retries) ? retries : 3;
     var options = {
         method: method,
-        headers: {
-            'Content-Type': 'application/json; charset=utf-8',
-            'Accept': 'application/json'
-        },
-        body: JSON.stringify(stringifyFunctions(body || {}))
+        headers: {},
+        body: stringifyFunctions(body),
+        json: true,
+        timeout: 5000
     };
     if (opts) {
@@ -56 +58 @@
     }
     return request(url, options)
-        .handle({
-            '-1': _.identity,
-            default: function(status,result) { return JSON.parse(result); }
-        });
+    .handle(function(status,body){
+        if ( _.isUndefined(body) ) {
+            if ( retries > 0) {
+                return couchRequest(method,url,body,opts,retries-1);
+            } else {
+                return new HTTPResult(-1,new Error("No response after several retries."));
+            }
+        } else {
+            return body;
+        }
+    });
 }