Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected NTLM message Type 3 - should respond with 401 #61

Open
bjowes opened this issue Dec 17, 2018 · 0 comments
Open

Unexpected NTLM message Type 3 - should respond with 401 #61

bjowes opened this issue Dec 17, 2018 · 0 comments

Comments

@bjowes
Copy link
Contributor

bjowes commented Dec 17, 2018

When express-ntlm receives an unexpected NTLM message Type 3 it responds with 500 (internal server error). It would be more helpful to the client if it responds with 401 (unauthorized), since this will trigger a proper client to re-authenticate.
I have tested this scenario against a Microsoft IIS, and it responds with a 401.

Rather simple change, but I will provide a PR for it nonetheless :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants