[Repoze-dev] [issue155] Support encoding of cookies on repoze.who cookie plugin

2010-07-13 Thread Nuno Teixeira
New submission from Nuno Teixeira tei...@gmail.com: I think that encoding cookies data could be useful. My usecase is related with SQLAlchemy that avoid the use of non-unicode strings on queries. -- files: r9728.diff messages: 426 nosy: nteixeira priority: feature status: unread title:

[Repoze-dev] [issue155] Support encoding of cookies on repoze.who cookie plugin

2010-07-13 Thread Tres Seaver
Tres Seaver tsea...@agendaless.com added the comment: The patch looks reasonable, but needs test coverage. Can you add tests for the encoding / decoding code paths? -- nosy: +tseaver status: unread - chatting __ Repoze Bugs b...@bugs.repoze.org

[Repoze-dev] [issue155] Support encoding of cookies on repoze.who cookie plugin

2010-07-13 Thread Nuno Teixeira
Nuno Teixeira tei...@gmail.com added the comment: Added tests for full coverage __ Repoze Bugs b...@bugs.repoze.org http://bugs.repoze.org/issue155 __Index: repoze/who/plugins/cookie.py

Re: [Repoze-dev] [issue155] Support encoding of cookies on repoze.who cookie plugin

2010-07-13 Thread Wichert Akkerman
On 2010-7-13 17:37, Nuno Teixeira wrote: -def __init__(self, cookie_name, cookie_path='/'): +def __init__(self, cookie_name, cookie_path='/', charset=None): Can we standardise on using 'encoding' as parameter name instead of 'charset' ? I think we use encoding everywhere else as well.