Tres Seaver <tsea...@agendaless.com> added the comment:

Thanks for adding tests.  I noticed that the tests did not actually confirm that
the codec had been used properly, which led me to find a bud in the main part
of the patch:  the charset must be used to encode the login and password 
*before*
applying the 'base54' codec.

I have applied the updated diff to the trunk for the upcoming 2.0b1 release.

----------
status: chatting -> resolved

__________________________________
Repoze Bugs <b...@bugs.repoze.org>
<http://bugs.repoze.org/issue155>
__________________________________
_______________________________________________
Repoze-dev mailing list
Repoze-dev@lists.repoze.org
http://lists.repoze.org/listinfo/repoze-dev

Reply via email to