Initialise middleware components automatically [#6]

Author: Tom Kirkpatrick

README.md

@@ -35,14 +35,13 @@ An 'access' [Operation Hook](https://docs.strongloop.com/display/public/LB/Opera
 
 4. Create a middleware.json file in your server folder (if you don't already have one).
 
-5. Enable the `loopback#context`, `loopback#token` and `loopback-component-group-access#user-context` middleware.
+5. Ensure that you enable the `loopback#context`, `loopback#token` middleware early in your middleware chain.
 
   ```json
   {
     "initial:before": {
       "loopback#context": {},
-      "loopback#token": {},
-      "loopback-component-group-access#user-context": {}
+      "loopback#token": {}
     },
   }
   ```

lib/index.js

@@ -2,6 +2,8 @@
 
 const debug = require('debug')('loopback:componenet:access');
 const AccessUtils = require('./utils');
+const accessLogger = require('./middleware/access-logger');
+const userContext = require('./middleware/user-context');
 
 module.exports = function loopbackComponentAccess(app, options) {
   debug('initializing component');
@@ -13,6 +15,10 @@ module.exports = function loopbackComponentAccess(app, options) {
     throw new Error('loopback-component-access-groups requires loopback 2.0 or newer');
   }
 
+  // Initialize middleware
+  app.middleware('initial:before', userContext());
+  app.middleware('routes:before', accessLogger());
+
   // Initialise helper class.
   const accessUtils = new AccessUtils(app, options);
 

lib/middleware/access-logger.js

@@ -3,6 +3,7 @@
 const debug = require('debug')('loopback:componenet:access:logger');
 
 module.exports = function accessLoggerMiddleware() {
+  debug('initializing access logger middleware');
   return function accessLogger(req, res, next) {
     if (req.accessToken) {
       debug('req: %s %s, token: %o', req.method, req.originalUrl, req.accessToken);

lib/middleware/user-context.js

@@ -5,6 +5,7 @@ const loopback = require('loopback');
 const Promise = require("bluebird");
 
 module.exports = function userContextMiddleware() {
+  debug('initializing user context middleware');
   // set current user to enable user access for remote methods
   return function userContext(req, res, next) {
     const loopbackContext = loopback.getCurrentContext();

test/fixtures/simple-app/server/middleware.json

@@ -2,17 +2,11 @@
   "initial:before": {
     "loopback#favicon": {},
     "loopback#context": {},
-    "loopback#token": {},
-    "../../../../lib/middleware/user-context": {}
+    "loopback#token": {}
   },
   "session": {},
   "auth": {},
   "parse": {},
-  "routes:before": {
-    "../../../../lib/middleware/access-logger": {
-      "paths": "/api/*"
-    }
-  },
   "routes": {
     "loopback#rest": {
       "paths": [