oglueck 2002/10/18 02:18:21
Modified: httpclient/src/java/org/apache/commons/httpclient
StatusLine.java
httpclient/src/test/org/apache/commons/httpclient
TestStatusLine.java
Log:
Fixed reason phrase parsing in status line
Revision Changes Path
1.3 +5 -5 jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/StatusLine.java
Index: StatusLine.java
===================================================================
RCS file: /home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/StatusLine.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- StatusLine.java 5 Sep 2002 04:48:43 -0000 1.2
+++ StatusLine.java 18 Oct 2002 09:18:21 -0000 1.3
@@ -152,7 +152,7 @@
//handle the Reason-Phrase
try {
if (to < length) {
- if (statusLine.endsWith("\n\r")) {
+ if (statusLine.endsWith("\r\n")) {
this.reasonPhrase = statusLine.substring(to+1, length-2);
} else {
this.reasonPhrase = statusLine.substring(to+1);
1.2 +5 -5 jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestStatusLine.java
Index: TestStatusLine.java
===================================================================
RCS file: /home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestStatusLine.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- TestStatusLine.java 5 Sep 2002 00:21:57 -0000 1.1
+++ TestStatusLine.java 18 Oct 2002 09:18:21 -0000 1.2
@@ -114,7 +114,7 @@
assertEquals("Non Trouvé", statusLine.getReasonPhrase());
//its ok to end with a \n\r
- statusLine = new StatusLine("HTTP/1.1 404 Not Found\n\r");
+ statusLine = new StatusLine("HTTP/1.1 404 Not Found\r\n");
assertEquals("Not Found", statusLine.getReasonPhrase());
//this is valid according to the Status-Line BNF
--
To unsubscribe, e-mail: <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>
|