[Groonga-commit] groonga/express-droonga [master] Disable pretty print of REST API for JSON only for production environments

Back to archive index

YUKI Hiroshi null+****@clear*****
Mon Mar 4 11:23:42 JST 2013


YUKI Hiroshi	2013-03-04 11:23:42 +0900 (Mon, 04 Mar 2013)

  New Revision: b0f78deb9ab3dfccd65f4b75e50dbf762b9d3b1f
  https://github.com/groonga/express-droonga/commit/b0f78deb9ab3dfccd65f4b75e50dbf762b9d3b1f

  Log:
    Disable pretty print of REST API for JSON only for production environments

  Modified files:
    lib/frontend/dashboard-handler.js
    lib/frontend/rest-adaptor.js

  Modified: lib/frontend/dashboard-handler.js (+1 -1)
===================================================================
--- lib/frontend/dashboard-handler.js    2013-03-04 10:58:53 +0900 (2820e03)
+++ lib/frontend/dashboard-handler.js    2013-03-04 11:23:42 +0900 (2c7ddd6)
@@ -19,7 +19,7 @@ exports.register = function(application, params) {
     application.use(prefix, express.static(path.join(__dirname, '..', 'public')));
   });
 
-  application.configure('development', function(){
+  application.configure('development', function() {
     application.use(prefix, express.errorHandler());
   });
 

  Modified: lib/frontend/rest-adaptor.js (+7 -2)
===================================================================
--- lib/frontend/rest-adaptor.js    2013-03-04 10:58:53 +0900 (4f0704a)
+++ lib/frontend/rest-adaptor.js    2013-03-04 11:23:42 +0900 (2b54940)
@@ -23,7 +23,7 @@ function createHandler(params) {
         if (error) {
           var body = envelope && envelope.body || null;
           response.contentType('application/json');
-          response.send(JSON.stringify(body), error);
+          response.send(body, error);
         } else {
           var body = envelope.body;
           if (definition.toClient) {
@@ -31,7 +31,7 @@ function createHandler(params) {
             body = result[1];
           }
           response.contentType('application/json');
-          response.send(JSON.stringify(body), 200);
+          response.send(body, 200);
         }
       },
       { timeout: timeout }
@@ -87,5 +87,10 @@ exports.register = function(application, params) {
                               definition: definition,
                               handler:    handler });
   });
+
+  application.configure('production', function() {
+    application.set('json spaces', -1); // disable pretty print
+  });
+
   return registeredCommands;
 }
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index