Only escape necessary chars in cookie value.

jjlee / mechanize

Stateful programmatic web browsing in Python, after Andy Lester's Perl module WWW::Mechanize .

http://wwwsearch.sourceforge.net/mechanize/

618 stars 121 forks source link

Only escape necessary chars in cookie value. #79

Closed cluther closed 7 years ago

cluther commented 11 years ago

I ran into the problem this patch fixes when using twill. A server was sending cookie's where the value was an equals (=) character. This caused mechanize to send an escaped cookie back to the server, which it was then unable to understand. So authentication failed.

Thanks!

cluther commented 7 years ago

Four years of silence makes me think that maybe pull requests aren't accepted. Closing for cleanliness.

jamesbroadhead commented 7 years ago

Thank you for your contribution to mechanize!

Following the process in #117, future work on mechanize will be occurring here: https://github.com/python-mechanize/mechanize.

Please re-file your PR there (where it will get attention, and hopefully merged)