Pull requests

#42 Merged
Repository
BoyanAlexandrov
Branch
default
Repository
cherrypy
Branch
default

Issue with cherrypy.lib.auth.check_auth()

Author
  1. BoyanAlexandrov
Reviewers
Description

The behavior of cherrypy.lib.auth.check_auth() didn't seem right, and I tracked it down to _checkBasicResponse in cherrypy.lib.httpauth.

  • Learn about pull requests

Comments (3)

  1. Joel Rivera

    I think you are right, _checkBasicResponse is not handling the case when is encrypt is None, but how are you using the check_auth function?. I mean at line 14 of auth.py is a validation in case that the encrypt is a non-True value. Have you consider to use auth_basic.py