Re: several fixes

* Terje Bless wrote:
>Granted that it might be usefull, but I don't like it. Maybe if you managed
>to get lstein to handle it transparently in CGI.pm[1], but I really (really
>really) don't want to parse CGI in core. One reason for that is that I
>don't really feel certain what the correct interpretation is in various
>cases. Another is that "/referer" is a kludge left in for backwards
>compatibility; it should have been moved to a CGI parameter or made into a
>"magic" URL. I certainly don't want to _add_ features to it.

Well, you can offer something like
http://validator.w3.org/check?uri=referer&ss= if you feel better with
that :-)

>>>http://foo.com --> http://foo.com/
>>
>>Yes, that trailing slash can be omittet, I don't see any good reason to
>>add the slash here,
>
>Sold! The extra slash is history. :-)

You've forgotten

| --- check.old   Wed Apr 18 05:39:23 2001
| +++ check       Wed Apr 18 05:39:52 2001
| @@ -325,7 +325,7 @@
| 
|  #
|  # Print different things if we got redirected or had a file upload.
| -if ($File->{URI} eq $q->param('uri')) {
| +if (URI::eq($File->{URI}, $q->param('uri'))) {
|    print ' ' x 4, qq(<li><a href="$uri_def_uri">URI</a>: );
|    print '<a href="', ent($File->{URI}), '">', ent($File->{URI}), qq(</a>\n);
|  } elsif ($q->param('uploaded_file')) {

(quoted to prevent line wrapping) in revision 1.96.
-- 
Björn Höhrmann { mailto:bjoern@hoehrmann.de } http://www.bjoernsworld.de
am Badedeich 7 } Telefon: +49(0)4667/981028 { http://bjoern.hoehrmann.de
25899 Dagebüll { PGP Pub. KeyID: 0xA4357E78 } http://www.learn.to/quote/

Received on Tuesday, 17 April 2001 23:42:33 UTC