Use promises if cb isn't passed in

[clarkie]

• supports hapi v17

I'm a little out of my depth on this one but this fixed the issue I was having when using https://github.com/salzhrani/hapi-auth-jwt2/tree/v-17 with hapi v17 where I was getting the following error:

180223/145803.900, [error] message: Cannot match against 'undefined' or 'null'., stack: TypeError: Cannot match against 'undefined' or 'null'.
    at Object.authenticate (/Users/clarkie/code/tojs/server/node_modules/hapi-auth-jwt2/lib/index.js:126:115)
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:228:7)

Let me know if I need to do any more to this to handle the other cases.

[nemophrost]

Might be good to return the whole key here?
return resolve(Object.assign({}, key, { key: key.publicKey || key.rsaPublicKey }))

[L42y]

Let's give this issue some love, so people can migrate to hapi v17 with ease.

[smessimer]

Any word on this? I'm trying to integrate Auth0 into a Hapi 17 project and am seeing this issue.

[aneilbaboo]

Fresh off the presses. A promisified version here: https://github.com/aneilbaboo/node-jwks-rsa-promisified

@clarkie @smessimer - let me know if it works for you.

[jl-dos]

Check out https://github.com/auth0/node-jwks-rsa/tree/master/examples/hapi-demo and the comments about the different function needed in the base module for v17 support. That plus the complete:true flag should get it all working.

[michalczukm]

@jl-dos indeed, it works! Thanks for pointing it!
btw Its terribly hidden 😄

For other devs hitting this ticket:
I described how to use it in SO answer: https://stackoverflow.com/a/51995573/2029818