[PATCH] avoid \xZZ in mirrorlist urls

[Matt Domsch]

On Wed, May 12, 2010 at 08:56:12PM -0500, Matt Domsch wrote:
> > Hey is there a way to test this on staging first to make sure it grabs
> > the URLs. RewriteRules's make my head hurt and I get things backwards
> > all the time.
> 
> In staging, I can test whether or not \x gets caught.  I can't test
> whether it'll catch the ones that are causing the problems from
> outside users though.  (If I could, I would have been able to find and
> fix the root cause, which clearly this isn't).
> 
> I'm open to other ideas.  Perhaps what I see in the URL logs isn't
> what's actually being sent?  I don't know...

Here's the symptom.  admin.fp.o/status/app01

16-0					    30536   0/1113/1113	W   18.42   11020  0   0.0   10.07  10.07	192.168.1.7		     app01.phx2.fedoraproject.org		     GET /mirrorlist?repo=\xc2\xadfedora-8&arch=i386 HTTP/1.0
63-0					    30713   0/2151/2151	W   38.47   3819 0 0.0 21.13 21.13  192.168.1.7 app01.phx2.fedoraproject.org GET /mirrorlist?repo=\xc2\xadfedora-8&arch=i386 HTTP/1.0

In both cases, the jobs are in W state from apache's POV (so
responding to the client request), see how long they've been sitting
there (11020 and 3819 seconds respectively), and the format of the
query args.  I send the same thing via wget, and it succeeds, no failure.


-- 
Matt Domsch
Technology Strategist
Dell | Office of the CTO