+1 (even later) > -----Original Message----- > From: Vincent Massol [mailto:[EMAIL PROTECTED]] > Sent: Saturday, December 21, 2002 12:22 PM > To: 'Cactus Developers List' > Subject: RE: [GUMP] Build Failure - jakarta-cactus-sample-servlet-13 > > > +1 (a bit late ;-)). > > Thanks > -Vincent > > > -----Original Message----- > > From: Christopher Lenz [mailto:[EMAIL PROTECTED]] > > Sent: 19 December 2002 12:19 > > To: Cactus Developers List > > Subject: Re: [GUMP] Build Failure - jakarta-cactus-sample-servlet-13 > > > > Vincent Massol wrote: > > > ---------------------------------------------------- > > > This email is autogenerated from the output from: > > > <http://cvs.apache.org/builds/gump/2002-12-18/jakarta-cactus-sample- > > servlet-13.html> > > > ---------------------------------------------------- > > [snip] > > > [java] [junit] Testcase: testReceiveCookie took 0.048 sec > > > [java] [junit] Caused an ERROR > > > [java] [junit] Invalid port: -1 > > > [java] [junit] java.lang.IllegalArgumentException: Invalid > > port: -1 > > > [java] [junit] at > > > org.apache.commons.httpclient.cookie.CookieSpecBase.parse(CookieSpecBase > .j > > ava:167) > > > [java] [junit] at > > > org.apache.commons.httpclient.cookie.CookieSpecBase.parse(CookieSpecBase > .j > > ava:265) > > > [java] [junit] at > > org.apache.commons.httpclient.Cookie.parse(Cookie.java:703) > > > [java] [junit] at > > org.apache.commons.httpclient.Cookie.parse(Cookie.java:631) > > > [java] [junit] at > > > org.apache.cactus.WebResponse.getCookies(WebResponse.java;org/apache/cac > tu > > s/util/log/LogAspect.aj(1k):292) > > > [java] [junit] at > > > org.apache.cactus.WebResponse.getCookie$ajcPostAround15(WebResponse.java > ;o > > rg/apache/cactus/util/log/LogAspect.aj(1k):223) > > > [java] [junit] at > > > org.apache.cactus.WebResponse.getCookie(WebResponse.java;org/apache/cact > us > > /util/log/LogAspect.aj(1k):1240) > > > [java] [junit] at > > org.apache.cactus.sample.TestSampleServlet.endReceiveCookie(Unknown > > Source) > > > [java] [junit] at > > sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > > > [java] [junit] at > > > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.jav > a: > > 39) > > > [java] [junit] at > > > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessor > Im > > pl.java:25) > > > [java] [junit] at > > > org.apache.cactus.AbstractWebTestCase.callGenericEndMethod(AbstractWebTe > st > > Case.java:182) > > > [java] [junit] at > > > org.apache.cactus.AbstractWebTestCase.callEndMethod(AbstractWebTestCase. > ja > > va:235) > > > [java] [junit] at > > > org.apache.cactus.AbstractWebTestCase.runGenericTest(AbstractWebTestCase > .j > > ava:263) > > > [java] [junit] at > > > org.apache.cactus.AbstractWebTestCase.runTest(AbstractWebTestCase.java:3 > 25 > > ) > > > [java] [junit] at > > org.apache.cactus.AbstractTestCase.runBare(AbstractTestCase.java:255) > > > > > > [java] [junit] Testcase: testReceiveCookie > > > > The problem here is that HttpClient seems to have become more strict > > when parsing Cookies. Cactus is specifying a port of "-1", which is > > obviously illegal, and HttpClient is (IMHO) correctly complaining > about > > that. > > > > A possible fix would be the following: in o.a.c.ServletURL#getPort(), > > we'll default to -1 when no port is specified (line 356). Instead, we > > should probably default to the default HTTP port 80 (or 443 if the > > protocol is HTTPS). *If* the port is specified but invalid (i.e. a > > NumberFormatException is thrown), we'd still return -1. > > > > I've tried the change on my local workspace, and the tests all run > > successfully. I also think that the behavior described above would be > > more correct than the current behavior. > > > > What do you think? Any objections? > > > > -- > > Christopher Lenz > > /=/ cmlenz at gmx.de > > > > > > -- > > To unsubscribe, e-mail: <mailto:cactus-dev- > > [EMAIL PROTECTED]> > > For additional commands, e-mail: <mailto:cactus-dev- > > [EMAIL PROTECTED]> > > > > -- > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> >
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>