Merge pull request #66 from digital-land/fixPageViewLogging

Fix page view logging

src/controllers/pageController.js

@@ -8,8 +8,8 @@ class PageController extends Controller {
     super.locals(req, res, callback)
   }
 
-  async get (req, res, next) {
-    await logPageView(this.options.route, req.sessionID, req.ip)
+  get (req, res, next) {
+    logPageView(this.options.route, req.sessionID, req.ip)
     super.get(req, res, next)
   }
 }

src/controllers/uploadController.js

@@ -8,7 +8,7 @@ import logger from '../utils/logger.js'
 class UploadController extends PageController {
   apiRoute = config.api.url + config.api.validationEndpoint
 
-  async get (req, res, next) {
+  get (req, res, next) {
     req.form.options.validationError = this.validationErrorMessage
     super.get(req, res, next)
   }

src/utils/logging.js

@@ -1,13 +1,13 @@
 import logger from '../utils/logger.js'
 import hash from '../utils/hasher.js'
 
-const logPageView = async (route, sessionID, ipAddress) => {
+const logPageView = (route, sessionID, ipAddress) => {
   logger.info({
     type: 'PageView',
     pageRoute: route,
     message: `page view occurred for page: ${route}`,
-    sessionId: await hash(sessionID),
-    ipAddress: await hash(ipAddress)
+    sessionId: hash(sessionID),
+    ipAddress: hash(ipAddress)
   })
 }