several fixes

Hi,

the attached patch should deal with the following issues

  * don't claim to be redirected from http://host or http://host:80/ to
    http://host/ -> uses URI::eq()
  * un-break /referer;ss etc. -> lets CGI.pm parse the trailing part
  * valid HTML and use of "constants" for redirect_to_home_page()
  * the line

      $q->param('uri', $q->param('uri') . '/')
        unless $q->param('uri') =~ m(/);

    is useless, $q->param('uri') always contains a '/' or the '/' isn't
    needed anywhere -> deleted

I currently have no Text::Iconv installed, so i cannot test these
changes...

other issues

  * prevent caching of /referer documents
    -> what about using a HTTP::Headers object for the header instead of
       a simple string?

  * _never_ redirect to homepage if the user requested some action like
    http://validator.w3.org/check?uri=foo, display error message instead

  * use $q->redirect for redirects

  * what about using a XML validator without the limitations of nsgmls
    to validate XML documents?

Oh yes, one more typo

--- todo.old    Tue Apr 17 06:19:43 2001
+++ todo.html   Tue Apr 17 22:08:38 2001
@@ -60,7 +60,7 @@

   <li> always display source code if there are errors?
     (or maybe just show a few lines of context for each error?)
-    Or add a "show source iff errors" option?
+    Or add a "show source if errors" option?

   <li> allow twiddling of options from the results page
-- 
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 16:17:22 UTC