search

johnbrett / hapi-auth-bearer-token

Simple Bearer authentication scheme plugin for hapi, accepts token by Header, Cookie or Query parameter.
MIT License
218 stars 46 forks source link

Chaining multiple queryToken strategies does not work #124

Closed AdriVanHoudt closed 6 years ago

AdriVanHoudt commented 7 years ago

Since the query token get's deleted here: https://github.com/johnbrett/hapi-auth-bearer-token/blob/master/lib/index.js#L55 the next strategy looking for it simply can't find it. Not sure how to fix this but I think the easiest is to delete it only just before a successful auth here https://github.com/johnbrett/hapi-auth-bearer-token/blob/master/lib/index.js#L94

johnbrett commented 7 years ago

I'll take a look at this on Monday @AdriVanHoudt! Sorry for delay, I'm way behind on OSS, thanks for opening this 👍

AdriVanHoudt commented 7 years ago

np!

johnbrett commented 6 years ago

closed via https://github.com/johnbrett/hapi-auth-bearer-token/pull/156