Changeset 527 for Dev/trunk/src/server/api/util.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/api/util.js (modified) (11 diffs)

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

@@ -34 +34 @@
     };
 
+    var handleException = exports.handleException = function(error) {
+        return new HTTPResult(500,{
+            reason: error.message,
+            filename: error.filename,
+            lineNumber: error.lineNumber
+        });
+    };
     var handleUnknownResponse = exports.handleUnknownResponse = function(status,error) {
         return new HTTPResult(500,{error: error.reason});
@@ -77 +84 @@
         return couch.get(url)
         .handle({
+            '-1': _.identity,
             200: handleRowValues,
             404: function() { return {error: "Cannot find collection of type "+type}; },
@@ -89 +97 @@
         return couch.get(id,opts)
         .handle({
+            '-1': _.identity,
             200: function(doc){
                 if ( doc.type !== type ) {
@@ -114 +123 @@
             return couch.put(id,doc,opts)
             .handle({
+                '-1': _.identity,
                 201: function(res){
                     doc._id = res.id;
@@ -133 +143 @@
             return couch.delete(id,opts)
             .handle({
+                '-1': _.identity,
                 200: identity,
                 409: function(error) {
@@ -149 +160 @@
             return couch.post(doc)
             .handle({
+                '-1': _.identity,
                 201: function(response) {
                     doc._id = response.id;
@@ -164 +176 @@
         return function(req,res) {
             getDocumentsOfType(type)
+            .handle({'-1': handleException})
             .handle(res.send.bind(res));
         };
@@ -173 +186 @@
             getDocument(id,rev,type)
             .handle({
+                '-1': function(result) {
+                    handleException(result)
+                    .handle(res.send.bind(res));
+                },
                 200: function(doc){
                     res.set({
@@ -189 +206 @@
             putDocument(id,rev,type,doc)
             .handle({
+                '-1': function(result) {
+                    handleException(result)
+                    .handle(res.send.bind(res));
+                },
                 201: function(doc) {
                     res.set({
@@ -204 +225 @@
             var rev = etags.parse(req.header('If-Match'))[0] || (doc && doc._rev);
             deleteDocument(id,rev)
+            .handle({'-1': handleException})
             .handle(res.send.bind(res));
         };
@@ -212 +234 @@
             postDocument(type,doc)
             .handle({
+                '-1': function(result) {
+                    handleException(result)
+                    .handle(res.send.bind(res));
+                },
                 201: function(doc) {
                     res.set({