Bug 24001 - LOCK: parsing of Timeout request header
Summary: LOCK: parsing of Timeout request header
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 4
Classification: Unclassified
Component: Servlets:WebDAV (show other bugs)
Version: 4.1.27
Hardware: Other other
: P3 major with 1 vote (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL: http://greenbytes.de/tech/webdav/rfc2...
Keywords:
Depends on:
Blocks:
 
Reported: 2003-10-22 13:08 UTC by Julian Reschke
Modified: 2004-11-16 19:05 UTC (History)
0 users


Attachments
Patch for TC5 to fix this bug (1.05 KB, patch)
2003-11-16 13:56 UTC, Mark Thomas 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Julian Reschke 2003-10-22 13:08:59 UTC 
"Timeout" header parser doesn't seem to support multiple, comma-separated values:

For instance:

LOCK ...
Timeout: Second-2,Infinite

yields a server error (500), stack trace:

java.lang.NumberFormatException: 2,Infinite
        at java.lang.Integer.parseInt(Integer.java:423)
        at java.lang.Integer.<init>(Integer.java:549)
        at org.apache.catalina.servlets.WebdavServlet.doLock(WebdavServlet.java:
875)
        at org.apache.catalina.servlets.WebdavServlet.service(WebdavServlet.java
:320)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:853)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(Appl
icationFilterChain.java:247)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationF
ilterChain.java:193)
        at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperV
alve.java:256)
        at org.apache.catalina.core.StandardPipeline$StandardPipelineValveContex
t.invokeNext(StandardPipeline.java:643)
        at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.jav
a:480)
Comment 1 Mark Thomas 2003-11-16 13:56:39 UTC 
Created attachment 9131 [details]
Patch for TC5 to fix this bug
Comment 2 Mark Thomas 2003-11-20 21:20:36 UTC 
A combined patch for this (and 23999, 24005) is attached to bug23999.
Comment 3 Mark Thomas 2003-12-10 21:39:17 UTC 
The necessary changes have been committed to CVS and will be included in the 
next release.