Fix composite auth OR - Githubissues

fastify / fastify-auth

Run multiple auth functions in Fastify

Other

341 stars 56 forks source link

Fix composite auth OR #224

Closed yakovenkodenis closed 6 months ago

yakovenkodenis commented 6 months ago

This fixes issue described in #223

Checklist

[x] run npm run test and npm run benchmark
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin and the Code of conduct

isnifer commented 6 months ago

I completed to create repro for #223, I will check this version locally immediately

isnifer commented 6 months ago

@yakovenkodenis thank you for your time, I can confirm that your fix is working.

On this screenshot I made a request and passed it through preHandler decorated via @fastify/auth. On the left side 4.4.0, in the middle 4.5.0, on the right side fixed version from Denis.

mcollina commented 6 months ago

@Eomm could you take a look as well?