Revert "change API_KEYS variable to be comma separated string as we'r…

…e not using the JSON data at the mo and JSON causes issues in docker-compose.yml"

This reverts commit c24d919.

Gruntfile.js

@@ -50,7 +50,7 @@ module.exports = function (grunt) {
     },
     env: {
       test: {
-        API_KEYS: '12345'
+        API_KEYS: '{"12345": "test-user"}'
       }
     }
   });

lib/server.js

@@ -21,7 +21,7 @@ const plugins = [
 
 let authKeys;
 if (process.env.API_KEYS) {
-  authKeys = process.env.API_KEYS.split(',');
+  authKeys = JSON.parse(process.env.API_KEYS);
 } else {
   throw new Error('Environment variable API_KEYS not set');
 }

lib/token_auth_strategy.js

@@ -1,9 +1,9 @@
 'use strict';
 
-module.exports = function (keys) {
+module.exports = function (keyData) {
   return (token, callback) => {
-    if (keys.indexOf(token) >= 0) {
-      return callback(null, true, {token});
+    if (keyData[token]) {
+      return callback(null, true, {name: keyData[token], token});
     }
 
     callback(null, false);

test/token_auth_strategy_tests.js

@@ -7,7 +7,7 @@ describe('token authenticator', () => {
   let authenticator;
 
   before(() => {
-    authenticator = authenticatorFactory('123456');
+    authenticator = authenticatorFactory({123456: 'bigwednesday.io'});
   });
 
   it('returns true when token is found', () => {
@@ -20,7 +20,7 @@ describe('token authenticator', () => {
   it('returns credentials when token is found', () => {
     authenticator('123456', (err, valid, credentials) => {
       expect(err).not.to.be.ok;
-      expect(credentials).to.deep.equal({token: '123456'});
+      expect(credentials).to.deep.equal({name: 'bigwednesday.io', token: '123456'});
     });
   });