W3C home > Mailing lists > Public > www-validator@w3.org > January 2007

Re: Cyclic redirect breaks Konquerors 'Validate HTML'

From: Ville Skyttä <ville.skytta@iki.fi>
Date: Tue, 23 Jan 2007 23:30:02 +0200
To: Benjamin Niemann <pink@odahoda.de>
Cc: www-validator@w3.org, Olivier Thereaux <ot@w3.org>
Message-Id: <200701232330.03254.ville.skytta@iki.fi>

On Tuesday 23 January 2007 21:32, Benjamin Niemann wrote:

> when I use Konquerors 'Validate HTML' function, the browsers tries to open
> e.g. <http://validator.w3.org/check/?uri=http%3A//www.odahoda.de/>, which
> redirects to the same URL over and over again until the browser aborts
> with 'Found a cyclic link..'.
> The fault is probably on Konqis side ('check/?uri=' instead
> of 'check?uri='), but it worked as intended before and I haven't changed
> the config of Konqi nor have I updated any packages recently.

Kind of off topic, but while I suppose Konqueror is not strictly speaking 
wrong in passing through "/" in ?uri=http%3A//... as is, in this case it 
would be better off encoding it as %2F, like ?uri=http%3A%2F%2F...  It 
doesn't make any difference wrt. the issue at hand though.

> It's easy to fix the URL that Konqi uses, but the cyclic redirect qualifies
> as a bug IMHO. Perhaps this has been introduced by the recent CGI.pm
> update?

Incidentally, while digging into the issue that prompted the CGI.pm update, I 
ran into something similar locally (though I used a pretty pathological case 
to get it to happen, much more so than your example, so I wasn't that worried 
about it and didn't check if it happened with the old version too).

Anyway, found one thing that could cause it in the new version and submitted a 
patch upstream: http://rt.cpan.org/Public/Bug/Display.html?id=24479

Olivier, if you feel like it, patching CGI.pm with the patch attached to the 
rt.cpan.org report could be worth trying out.  To me it looks safe 
and "obviously correct", but I'm biased ;)
Received on Tuesday, 23 January 2007 21:30:10 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 1 March 2016 14:17:50 UTC