rest middleware: clean up context config

Modify `loopback.rest()` to read the configuration for
`loopback.context` from `app.get('remoting')`, which is the approach
used for all other configuration options related to the REST transport.

Author: Miroslav Bajtoš

lib/middleware/rest.js

@@ -22,17 +22,9 @@ module.exports = rest;
  * @header loopback.rest()
  */
 
-function rest(options) {
-  options = options || {};
+function rest() {
   var tokenParser = null;
   var contextHandler = null;
-  if (options.context) {
-    var contextOptions = options.context;
-    if(typeof contextOptions !== 'object') {
-      contextOptions = {};
-    }
-    contextHandler = loopback.context(contextOptions);
-  }
   return function restApiHandler(req, res, next) {
     var app = req.app;
     var handler = app.handler('rest');
@@ -44,9 +36,16 @@ function rest(options) {
     }
 
     var handlers = [];
-    if (options.context) {
+    var remotingOptions = app.get('remoting') || {};
+
+    var contextOptions = remotingOptions.context;
+    if (contextOptions !== false && !contextHandler) {
+      if (typeof contextOptions !== 'object')
+        contextOptions = {};
+      contextHandler = loopback.context(contextOptions);
       handlers.push(contextHandler);
     }
+
     if (app.isAuthEnabled) {
       if (!tokenParser) {
         // NOTE(bajtos) It would be better to search app.models for a model
@@ -66,5 +65,3 @@ function rest(options) {
     }, next);
   };
 }
-
-

test/rest.middleware.test.js

@@ -136,7 +136,7 @@ describe('loopback.rest', function() {
     }
 
     it('should enable context using loopback.context', function(done) {
-      app.use(loopback.context({enableHttpContext: true}));
+      app.use(loopback.context({ enableHttpContext: true }));
       app.enableAuth();
       app.use(loopback.rest());
 
@@ -145,7 +145,8 @@ describe('loopback.rest', function() {
 
     it('should enable context with loopback.rest', function(done) {
       app.enableAuth();
-      app.use(loopback.rest({context: {enableHttpContext: true}}));
+      app.set('remoting', { context: { enableHttpContext: true } });
+      app.use(loopback.rest());
 
       invokeGetToken(done);
     });