Skip to content
This repository has been archived by the owner on Feb 14, 2018. It is now read-only.

Basic auth does not handle passwords with colons #3

Open
lucap opened this issue Jan 30, 2012 · 0 comments
Open

Basic auth does not handle passwords with colons #3

lucap opened this issue Jan 30, 2012 · 0 comments

Comments

@lucap
Copy link

lucap commented Jan 30, 2012

Basic auth does not allow for passwords containing one or more colon (:) characters.

For example:


> > > basic_auth, basic_pass = auth_decoded.split(':', 2)
> > > Traceback (most recent call last):
> > >   File "<stdin>", line 1, in <module>
> > > ValueError: too many values to unpack
> > > 
> > > ```
> > > ```

Proposed fix:
```basic_auth, basic_pass = auth_decoded.split(':', 1)

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

No branches or pull requests

1 participant