Re: [PATCH] (X)HTML-ouput in proxy_util.c

[Erik Abele]

  > Erik Abele wrote:
  >
  >> While browsing through the source I discovered a place in
  >> proxy_util.c (line 612), where the output isn't XHTMLized so far.
  >>
  >> Since all other server-generated output and the error documents are
  >> valid XHTML, I think
  >> we should change this too. Attached a patch for review please. Are
  >> there any objections to not apply this? I don't think so...:-)

wuups, sorry, I forgot to attach the patch :-)

Erik

Index: proxy_util.c
===================================================================
RCS file: /home/cvs/httpd-2.0/modules/proxy/proxy_util.c,v
retrieving revision 1.97
diff -u -r1.97 proxy_util.c
--- proxy_util.c        15 Jul 2002 08:05:10 -0000      1.97
+++ proxy_util.c        25 Aug 2002 14:07:36 -0000
@@ -612,13 +612,13 @@
     apr_table_setn(r->notes, "error-notes",
        apr_pstrcat(r->pool, 
                "The proxy server could not handle the request "
-               "<EM><A HREF=\"", ap_escape_uri(r->pool, r->uri),
+               "<em><a href=\"", ap_escape_uri(r->pool, r->uri),
                "\">", ap_escape_html(r->pool, r->method),
                "&nbsp;", 
-               ap_escape_html(r->pool, r->uri), "</A></EM>.<P>\n"
-               "Reason: <STRONG>",
+               ap_escape_html(r->pool, r->uri), "</a></em>.<p>\n"
+               "Reason: <strong>",
                ap_escape_html(r->pool, message), 
-               "</STRONG>", NULL));
+               "</strong></p>", NULL));
 
     /* Allow "error-notes" string to be printed by ap_send_error_response() */
     apr_table_setn(r->notes, "verbose-error-to", apr_pstrdup(r->pool, "*"));