Re: doctype declaration for generated html code

From: PsychoTekk .de <psychotekk@dont-contact.us>
Date: Sat, 13 Jul 2002 05:54:07 -0500

> Adding doctype to the error pages is well.. no more than adding it to
> the error pages..
that's right. i first thought of the ERR_SQUID_SIGNATURE (errorpage.c)
as a header _and_ footer but the header would just include
doctype, html and head since title might differ from err msg to err
msg.
i think it would be nice to include the doctype in squid's error
documents directly by default then.

> Adding doctype to the generated HTML pages such as FTP/Gopher
> directory listings needs patching of the source.
>
> Patches to both the error messages and src/ftp.c and src/gopher.c are
> welcome.

ftp.c, line 366:
    storeAppendPrintf(e, "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>\n");

gopher.c, line 245:
            "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>Server Return Nothing.</TITLE>\n"

line 278:
            "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>Gopher Index %s</TITLE></HEAD>\n"

line 293:
            "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>CSO Search of %s</TITLE></HEAD>\n"

line 309:
            strCat(outbuf, "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>CSO Search Result</TITLE></HEAD>\n"

line 312:
            strCat(outbuf, "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>Gopher Menu</TITLE></HEAD>\n"

errorpage.c, line 67:
            "<HR noshade size=\"1px\">\n"

(some browsers need the quotation marks to detach segments:
attribute="value")

cachemgr.c, line 238:
    printf("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>Cache Manager Interface</TITLE></HEAD>\n");

line 263:
    printf("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>Cache Manager Error</TITLE></HEAD>\n");

line 452:
            printf("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n<HTML><HEAD><TITLE>CacheMgr@%s: %s</TITLE></HEAD><BODY>\n",

i'm sorry if these patches are not in the usual patch format but
i'm not sure how to generate .patch files in CVS(?) format.

> > Furthermore it would be useful to define a </u>charset</u> (this
> > can be either done in the body of the html document by using a
> > meta tag
> > <b><META HTTP-EQUIV="Content-Type" CONTENT="text/html;
> > charset=ISO-8859-1"></b> or by adding the specific header
> > information.
> > <b>Content-Type: text/html; charset=ISO-8859-1</b>)
>
> I agree, but HTTP-EQUIV is not a good method. HTTP-EQUIV is intended > a a hook to be used by servers when determine the Content-Type header
> to return. A browser reading the HTTP-EQUIV directive in the HTML
> source is doing the wrong thing.
>
> Basically, there needs to be a squid.conf directive to set the
> content-type charset attribute of error pages, or error.c needs to
> be extended to parse the HTTP-EQUIV directive.
>
> Regards
> Henrik
>

it would be great to have a directive like apache's AddDefaultCharset
(though this is related to its TCN ability, in squid.conf it would
be something like err_charset)
i will look at that as soon as i have enough time again

regards,
 C.L.

-- 
__________________________________________________________
Sign-up for your own FREE Personalized E-mail at Mail.com
http://www.mail.com/?sr=signup


Save up to $160 by signing up for NetZero Platinum Internet service.
http://www.netzero.net/?refcd=N2P0602NEP8

Received on Sat Jul 13 2002 - 04:59:39 MDT

This archive was generated by hypermail pre-2.1.9 : Tue Dec 09 2003 - 16:15:48 MST