Re: [thredds] thredds Digest, Vol 30, Issue 18

• To: "thredds@xxxxxxxxxxxxxxxx" <thredds@xxxxxxxxxxxxxxxx>
• Subject: Re: [thredds] thredds Digest, Vol 30, Issue 18
• From: Ming Fu <Xiao.Fu@xxxxxxxxxxx>
• Date: Mon, 1 Aug 2011 01:08:33 +0000

The Tomcat version is 6.0.32.

-----Original Message-----
From: thredds-bounces@xxxxxxxxxxxxxxxx 
[mailto:thredds-bounces@xxxxxxxxxxxxxxxx] On Behalf Of 
thredds-request@xxxxxxxxxxxxxxxx
Sent: Saturday, 30 July 2011 4:00 AM
To: thredds@xxxxxxxxxxxxxxxx
Subject: thredds Digest, Vol 30, Issue 18

Send thredds mailing list submissions to
        thredds@xxxxxxxxxxxxxxxx

To subscribe or unsubscribe via the World Wide Web, visit
        http://mailman.unidata.ucar.edu/mailman/listinfo/thredds
or, via email, send a message with subject or body 'help' to
        thredds-request@xxxxxxxxxxxxxxxx

You can reach the person managing the list at
        thredds-owner@xxxxxxxxxxxxxxxx

When replying, please edit your Subject line so it is more specific
than "Re: Contents of thredds digest..."


thredds mailing list
thredds@xxxxxxxxxxxxxxxx
For list information or to unsubscribe,  visit: 
http://www.unidata.ucar.edu/mailing_lists/

Today's Topics:

   1. Bug report on Content-Length for HTTP Head Request (Ming Fu)
   2. Re: Bug report on Content-Length for HTTP Head Request
      (Ethan Davis)


----------------------------------------------------------------------

Message: 1
Date: Fri, 29 Jul 2011 02:29:41 +0000
From: Ming Fu <Xiao.Fu@xxxxxxxxxxx>
To: "thredds@xxxxxxxxxxxxxxxx" <thredds@xxxxxxxxxxxxxxxx>
Subject: [thredds] Bug report on Content-Length for HTTP Head Request
Message-ID:
        <74A8ED427A3CA0499D90AF67F69C59467F8B31CF@xxxxxxxxxxxxxxxx.internal>
Content-Type: text/plain; charset="us-ascii"

Hi All,

In thredds.servlet.ServletUtil.java, line 607:
  res.addHeader("Content-Length", Long.toString(contentLength));

The above line is not setting up "Content-Length" correctly for HTTP Head 
Request in Tomcat.
As the current implementation in Tomcat, using doGet method to handle HTTP Head 
request will be resulted in a NoBodyResponse to be returned.
The NoBodyResponse only accepts response.setContentLength method, otherwise, it 
always sets the Content-Length to 0.

Suggested fix is to override doHead() in thredds.servlet.FileServerServlet.java 
 and make it as a delegator to doGet() method.

Thanks,

Ming Fu
Software Engineer

Tasmanian Partnership for Advanced Computing
University of Tasmania

-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://mailman.unidata.ucar.edu/mailing_lists/archives/thredds/attachments/20110729/98831646/attachment.html>

------------------------------

Message: 2
Date: Fri, 29 Jul 2011 10:32:32 -0600
From: Ethan Davis <edavis@xxxxxxxxxxxxxxxx>
To: thredds@xxxxxxxxxxxxxxxx
Subject: Re: [thredds] Bug report on Content-Length for HTTP Head
        Request
Message-ID: <4E32E0A0.80406@xxxxxxxxxxxxxxxx>
Content-Type: text/plain; charset=windows-1252

Hi Ming Fu,

Thanks for reporting this problem. What version of Tomcat are you running?

I'm seeing it on a server running Tomcat 6.0.32. I've entered it into
our issue tracking system. You can follow it here if you'd like:

https://www.unidata.ucar.edu/jira/browse/TDS-199

Thanks,

Ethan

On 7/28/2011 8:29 PM, Ming Fu wrote:
> Hi All,
> 
> In thredds.servlet.ServletUtil.java, line 607:
> 
>   res.addHeader("Content-Length", Long.toString(contentLength));
> 
> The above line is not setting up ?Content-Length? correctly for HTTP
> Head Request in Tomcat.
> 
> As the current implementation in Tomcat, using doGet method to handle
> HTTP Head request will be resulted in a NoBodyResponse to be returned.
> 
> The NoBodyResponse only accepts response.setContentLength method,
> otherwise, it always sets the Content-Length to 0.
> 
> Suggested fix is to override doHead() in
> thredds.servlet.FileServerServlet.java  and make it as a delegator to
> doGet() method.
> 
> Thanks,
> 
> Ming Fu
> 
> Software Engineer
> 
> Tasmanian Partnership for Advanced Computing 
> University of Tasmania



End of thredds Digest, Vol 30, Issue 18
***************************************