Moved update-user-last-seen mw into api app

- Moved update-user-last-seen from shared to api as it is not shared (except within the API)
- This file is only used in one part of the app, this updates the code structure to reflect this
- This is one of many similar changes needed to make it easier to refactor to the existing setup

core/server/web/api/canary/admin/middleware.js

@@ -53,7 +53,7 @@ const notImplemented = function (req, res, next) {
 module.exports.authAdminApi = [
     auth.authenticate.authenticateAdminApi,
     auth.authorize.authorizeAdminApi,
-    shared.middlewares.updateUserLastSeen,
+    apiMw.updateUserLastSeen,
     apiMw.cors,
     shared.middlewares.urlRedirects.adminRedirect,
     shared.middlewares.prettyUrls,
@@ -67,7 +67,7 @@ module.exports.authAdminApi = [
 module.exports.authAdminApiWithUrl = [
     auth.authenticate.authenticateAdminApiWithUrl,
     auth.authorize.authorizeAdminApi,
-    shared.middlewares.updateUserLastSeen,
+    apiMw.updateUserLastSeen,
     apiMw.cors,
     shared.middlewares.urlRedirects.adminRedirect,
     shared.middlewares.prettyUrls,

core/server/web/api/middleware/index.js

@@ -1,4 +1,5 @@
 module.exports = {
     cors: require('./cors'),
+    updateUserLastSeen: require('./update-user-last-seen'),
     versionMatch: require('./version-match')
 };

core/server/web/api/v2/admin/middleware.js

@@ -50,7 +50,7 @@ const notImplemented = function (req, res, next) {
 module.exports.authAdminApi = [
     auth.authenticate.authenticateAdminApi,
     auth.authorize.authorizeAdminApi,
-    shared.middlewares.updateUserLastSeen,
+    apiMw.updateUserLastSeen,
     apiMw.cors,
     shared.middlewares.urlRedirects.adminRedirect,
     shared.middlewares.prettyUrls,
@@ -64,7 +64,7 @@ module.exports.authAdminApi = [
 module.exports.authAdminApiWithUrl = [
     auth.authenticate.authenticateAdminApiWithUrl,
     auth.authorize.authorizeAdminApi,
-    shared.middlewares.updateUserLastSeen,
+    apiMw.updateUserLastSeen,
     apiMw.cors,
     shared.middlewares.urlRedirects.adminRedirect,
     shared.middlewares.prettyUrls,

core/server/web/shared/middlewares/index.js

@@ -49,9 +49,5 @@ module.exports = {
 
     get urlRedirects() {
         return require('./url-redirects');
-    },
-
-    get updateUserLastSeen() {
-        return require('./update-user-last-seen');
     }
 };

test/unit/web/api/middleware/update-user-last-seen_spec.js

@@ -1,7 +1,7 @@
 const should = require('should');
 const sinon = require('sinon');
-const constants = require('../../../../core/server/lib/constants');
-const updateUserLastSeenMiddleware = require('../../../../core/server/web/shared/middlewares').updateUserLastSeen;
+const constants = require('../../../../../core/server/lib/constants');
+const updateUserLastSeenMiddleware = require('../../../../../core/server/web/api/middleware/update-user-last-seen');
 
 describe('updateUserLastSeenMiddleware', function () {
     afterEach(function () {