gvalkov / tornado-http-auth

Digest and basic authentication for Tornado
Apache License 2.0
19 stars 12 forks source link

fix too many values to unpack (expected 2) #2

Closed yoosofan closed 7 years ago

yoosofan commented 7 years ago

When a user put : (colon) on user name or password this packages produces this error. too many values to unpack (expected 2), line 210, in authenticate_user username, password = auth_data.split(':') It can be fixed simply by checking the number of colons if colon is prohibited in user name and password.

yoosofan commented 7 years ago

I think my pull request will solve the problem.

yoosofan commented 7 years ago

Sorry, I'm using Gholam Yaghoobi's computer. Therefore, his name is shown in commits.

gvalkov commented 7 years ago

Thank you for contributing :+1:

yoosofan commented 7 years ago

🙏