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

auth_digest returns 400 on wrong scheme, should be 401 #1719

Closed
jaraco opened this issue Jun 17, 2018 · 0 comments
Closed

auth_digest returns 400 on wrong scheme, should be 401 #1719

jaraco opened this issue Jun 17, 2018 · 0 comments

Comments

@jaraco
Copy link
Member

@jaraco jaraco commented Jun 17, 2018

The docs for the digest auth tool indicate that a 401 will be returned if the wrong scheme is supplied.

But tracing the code, it seems that when the scheme is wrong, a ValueError is raised and that's translated into a 400 response. I suspect the 401 response is the correct thing to do.

@jaraco jaraco closed this in 1d41828 Jun 17, 2018
jaraco added a commit that referenced this issue Jun 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.