[Twisted-web] fix nevow exception in args parsing

[Andrea Arcangeli]

On Sat, Sep 03, 2005 at 12:11:58PM +0100, Matt Goodall wrote:
> Matt Goodall wrote:
> > Andrea Arcangeli wrote:
> > 
> >>>I noticed an exception in nevow, for example with divmod.com/?=x=x=x.
> >>>
> >>>This is the fix, please apply this (or an equivalent one) to SVN:
> >>>
> >>>Index: Nevow/nevow/url.py
> >>>===================================================================
> >>>--- Nevow/nevow/url.py	(revision 1748)
> >>>+++ Nevow/nevow/url.py	(working copy)
> >>>@@ -21,7 +21,7 @@
> >>> def _uqf(query):
> >>>     for x in query.split('&'):
> >>>         if '=' in x:
> >>>-            yield tuple( [urllib.unquote_plus(s) for s in x.split('=')] )
> >>>+            yield tuple( [urllib.unquote_plus(s) for s in x.split('=')[:2]] )
> >>>         elif x:
> >>>             yield (urllib.unquote_plus(x), None)
> >>> unquerify = lambda query: list(_uqf(query))
> > 
> > 
> > I don't think that's correct - it throws away information. A better
> > solution is probably:
> > 
> >     yield tuple( [urllib.unquote_plus(s) for s in x.split('=', 1)] )
> > 
> > Do you agree? If so, I'll apply this fix later.
> 
> cgi.parse_qs agrees with me so I just committed my version of Andrea's
> fix (with a test). Thanks for highlighting the error Andrea.

You're welcome! I agree this is more correct (and hopefully all code
knows that "=" can be inside the argument ;).