jaredhanson / passport-http

HTTP Basic and Digest authentication strategies for Passport and Node.js.

Home Page:https://www.passportjs.org/packages/passport-http/?utm_source=github&utm_medium=referral&utm_campaign=passport-http&utm_content=about

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Shouldn't an authentication failure return status 401 as well?

sk91 opened this issue · comments

why this has been closed? is it still valid?

ok I guess this has been cloased because the callback you pass to the authorization method gets an error when there is an auth failure. In this case you can throw 401

@gnuton sorry, don't remember exactly why I closed it. I fixed it for my self and made a pull request:

#13

But it was ignored for more than 5 years ...