[v1.0.1] Corrected plugin interface

Author: antwhite

README.md

@@ -37,13 +37,6 @@ registry.register(
         }
       }
     }
-  },
-  err => {
-    if (err) {
-      console.log('plugin initialisation failed:', err);
-    } else {
-      console.log('jwt verify now available');
-    }
   }
 );
 
@@ -66,13 +59,6 @@ registry.register(
         }
       }
     }
-  },
-  err => {
-    if (err) {
-      console.log('plugin initialisation failed:', err);
-    } else {
-      console.log('jwt verify now available');
-    }
   }
 );
 

lib/verify.js

@@ -5,13 +5,13 @@ const defaultKeyStore = require('./defaultKeyStore');
 let keyStore;
 
 module.exports = {
-  register(options, callback) {
+  register(options, dependencies, next) {
     if (options && options.keyStore) {
       keyStore = options.keyStore;
     } else {
       keyStore = defaultKeyStore(options);
     }
-    return callback();
+    next();
   },
   execute(token, callback) {
     try {

lib/verify.test.js

@@ -5,15 +5,14 @@ const dataToTokenzie = {
   name: 'John Doe',
   sub: '1234567890'
 };
-const emptyCallback = () => {};
 
 describe('verify', () => {
   describe('when given a valid public key signed token', () => {
     const token = utils.getTokenSignedWithPrivateKey(dataToTokenzie);
 
     it('returns the verified token with no errors', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toBe(null);
         expect(verifiedToken).toMatchSnapshot();
@@ -26,7 +25,7 @@ describe('verify', () => {
 
     it('returns the verified token with no errors', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toBe(null);
         expect(verifiedToken).toMatchSnapshot();
@@ -39,7 +38,7 @@ describe('verify', () => {
 
     it('it returns an error with no token', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toMatchSnapshot();
         expect(verifiedToken).toBe(undefined);
@@ -59,7 +58,7 @@ describe('verify', () => {
           }
         }
       };
-      verify.register(options, emptyCallback);
+      verify.register(options, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toBe(null);
         expect(verifiedToken).toMatchSnapshot();
@@ -72,7 +71,7 @@ describe('verify', () => {
 
     it('it returns an error with no token', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toMatchSnapshot();
         expect(verifiedToken).toBe(undefined);
@@ -85,7 +84,7 @@ describe('verify', () => {
 
     it('it returns an error with no token', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toMatchSnapshot();
         expect(verifiedToken).toBe(undefined);
@@ -98,7 +97,7 @@ describe('verify', () => {
 
     it('it returns an error with no token', () => {
       const verify = require('./verify');
-      verify.register({ keys }, emptyCallback);
+      verify.register({ keys }, {}, () => {});
       verify.execute(token, (error, verifiedToken) => {
         expect(error).toMatchSnapshot();
         expect(verifiedToken).toBe(undefined);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "oc-plugin-jwt",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "OpenComponents Plugin for Verifying & Decoding JSON Web Tokens (JWT)",
   "main": "index.js",
   "scripts": {