Check chain of strategies on default server auth

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

Check chain of strategies on default server auth #129

Closed nakardo closed 7 years ago

nakardo commented 7 years ago

Currently when allowChaining is enabled, it looks for a chain of strategies on the route configuration. When a chain is set as default server auth, it breaks execution and returns 401.

P.S. Sorry for all the whitespace line changes -- I've created a new suite for these cases with a fresh server instance.

AdriVanHoudt commented 7 years ago

yeah you might want to look at it with the whitespace flag https://github.com/johnbrett/hapi-auth-bearer-token/pull/129/files?w=1

johnbrett commented 7 years ago

Closing this, will handle this PR in https://github.com/johnbrett/hapi-auth-bearer-token/pull/136, due to some git weirdness with the PR branch not being in this repo.

Thanks for this submission @nakardo - apologies for leaving this so late, I've fallen way behind in OSS!