Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Ricky Zhou
On 2010-05-12 11:05:33 PM, Matt Domsch wrote: > OK, having spent a few more hours, I don't need the RewriteRule. Yea! > > I just need to convert to unicode, and leave it in unicode. Malformed > requests fail lookups as would be expected and return the "you asked > for an invalid repo or arch" mes

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Toshio Kuratomi
On Wed, May 12, 2010 at 11:05:33PM -0500, Matt Domsch wrote: > On Wed, May 12, 2010 at 08:59:18PM -0500, Matt Domsch wrote: > > 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'

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Rick Elrod
On May 13, 2010, at 12:05 AM, Matt Domsch wrote: > > OK, having spent a few more hours, I don't need the RewriteRule. Yea! > > I just need to convert to unicode, and leave it in unicode. Malformed > requests fail lookups as would be expected and return the "you asked > for an invalid repo or ar

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Matt Domsch
On Wed, May 12, 2010 at 08:59:18PM -0500, Matt Domsch wrote: > 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. >

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread 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 tes

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Matt Domsch
> 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 outsi

Re: [PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Stephen John Smoogen
On Wed, May 12, 2010 at 5:41 PM, Matt Domsch wrote: > We are getting some mirrorlist requests with escape characters in them > such as \xe2 .  While I've taken steps to deal with these in the > mirrorlist code, at least one client makes such a request hourly, and > they are causing the mirrorlist

[PATCH] avoid \xZZ in mirrorlist urls

2010-05-12 Thread Matt Domsch
We are getting some mirrorlist requests with escape characters in them such as \xe2 . While I've taken steps to deal with these in the mirrorlist code, at least one client makes such a request hourly, and they are causing the mirrorlist WSGI process to spin. I can't recreate the failure, even usi