From: Alp Toker Date: Sun, 09 Jul 2006 09:41:58 GMT Subject: [PATCH] gitweb: Send XHTML as 'application/xhtml+xml' where possible Message-ID: <11524381182709-git-send-email-alp@atoker.com> "The 'text/html' media type [RFC2854] is primarily for HTML, not for XHTML. In general, this media type is NOT suitable for XHTML." This patch makes gitweb use content negotiation to conservatively send pages as Content-Type 'application/xhtml+xml' when the user agent explicitly claims to support it. It falls back to 'text/html' even if the user agent claims to support 'application/xhtml+xml' by means of a glob in order to work around an insidious Internet Explorer bug. Signed-off-by: Alp Toker --- gitweb/gitweb.cgi | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/gitweb/gitweb.cgi b/gitweb/gitweb.cgi index 3e2790c..3206435 100755 --- a/gitweb/gitweb.cgi +++ b/gitweb/gitweb.cgi @@ -290,7 +290,8 @@ sub git_header_html { } } } - print $cgi->header(-type=>'text/html', -charset => 'utf-8', -status=> $status, -expires => $expires); + my $content_type = ($cgi->http('HTTP_ACCEPT') =~ m/(,|;|\s|^)application\/xhtml\+xml(,|;|\s|$)/ && $cgi->Accept('application/xhtml+xml') ne 0) ? 'application/xhtml+xml' : 'text/html'; + print $cgi->header(-type=>$content_type, -charset => 'utf-8', -status=> $status, -expires => $expires); print < @@ -298,7 +299,7 @@ sub git_header_html { - + $title -- 1.4.1.gbe4c7