[Bug 54980] Setting 'cookies' attribute to 'false' in context still generate JSESSIONID cookie

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54980 Emac changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|---

[Bug 54928] IllegalStateException: Calling [asyncError()] is not valid for a request with Async state [COMPLETING]

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54928 --- Comment #7 from rstoyanc...@yahoo.com --- I can confirm the exception no longer appears in the logs. Thanks for fixing this! -- You are receiving this mail because: You are the assignee for the bug. ---

svn commit: r1485862 - /tomcat/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 21:37:23 2013 New Revision: 1485862 URL: http://svn.apache.org/r1485862 Log: Make this public to expand how the class can be used for unit tests. Modified: tomcat/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java Modified: tomcat/trunk/test/org/apach

svn commit: r1485855 - /tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 21:20:27 2013 New Revision: 1485855 URL: http://svn.apache.org/r1485855 Log: Remove duplicate code Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java URL: ht

svn commit: r1485854 - /tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 21:19:54 2013 New Revision: 1485854 URL: http://svn.apache.org/r1485854 Log: Trivial formatting Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java

svn commit: r1485847 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 20:50:05 2013 New Revision: 1485847 URL: http://svn.apache.org/r1485847 Log: Add some NPE protection Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java URL: http://svn

[Bug 54928] IllegalStateException: Calling [asyncError()] is not valid for a request with Async state [COMPLETING]

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54928 --- Comment #6 from Mark Thomas --- I think some other changes might have fixed this. Please can you re-test with the latest trunk. -- You are receiving this mail because: You are the assignee for the bug. ---

svn commit: r1485833 - /tomcat/trunk/test/org/apache/tomcat/util/net/TestSsl.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 19:22:50 2013 New Revision: 1485833 URL: http://svn.apache.org/r1485833 Log: Correctly init WebSocket to avoid NPE Modified: tomcat/trunk/test/org/apache/tomcat/util/net/TestSsl.java Modified: tomcat/trunk/test/org/apache/tomcat/util/net/TestSsl.java URL: http

[Bug 54387] Having servlets with same url should at least cause warning

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54387 --- Comment #6 from Violeta Georgieva --- (In reply to comment #4) > > Let's imagine that we have a Servlet annotated with @WebServlet("/an/url") > and a web.xml descriptor that define the same Servlet class with a different > logical name

[Bug 54387] Having servlets with same url should at least cause warning

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54387 --- Comment #5 from Edson Richter --- My understanding is that Scenario 1: ServletA -> /my/path/to/servlet ServletB -> /my/path/to/servlet Rule: same path to different servlets = failure. Scenario 2: ServletA -> /my/path/to/servlet1 S

[Bug 54387] Having servlets with same url should at least cause warning

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54387 --- Comment #4 from Loïc Albertin --- Hi Mark, I totally agree with the fact that an URL should be mapped to only one Servlet. But, I'd like to have your point of view and maybe the one of the Servlet EG about the following case. Let's im

[Bug 54980] Setting 'cookies' attribute to 'false' in context still generate JSESSIONID cookie

2013-05-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54980 Violeta Georgieva changed: What|Removed |Added Status|NEW |NEEDINFO OS|

svn commit: r1485720 - /tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 14:13:14 2013 New Revision: 1485720 URL: http://svn.apache.org/r1485720 Log: Only try a non-blocking read if non-blocking is being used Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java Modified: tomcat/trunk/java/org/apache/coy

svn commit: r1485660 - in /tomcat/trunk/java/org/apache/coyote/http11: InternalAprInputBuffer.java LocalStrings.properties

2013-05-23 Thread markt
Author: markt Date: Thu May 23 12:23:05 2013 New Revision: 1485660 URL: http://svn.apache.org/r1485660 Log: Better error message Modified: tomcat/trunk/java/org/apache/coyote/http11/InternalAprInputBuffer.java tomcat/trunk/java/org/apache/coyote/http11/LocalStrings.properties Modified: t

svn commit: r1485659 - /tomcat/trunk/java/org/apache/coyote/http11/LocalStrings.properties

2013-05-23 Thread markt
Author: markt Date: Thu May 23 12:21:16 2013 New Revision: 1485659 URL: http://svn.apache.org/r1485659 Log: Add missing message Modified: tomcat/trunk/java/org/apache/coyote/http11/LocalStrings.properties Modified: tomcat/trunk/java/org/apache/coyote/http11/LocalStrings.properties URL: http

buildbot success in ASF Buildbot on tomcat-trunk

2013-05-23 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-trunk/builds/4361 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-vm_ubuntu Build Reason: scheduler Build Source

svn commit: r1485612 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 08:31:37 2013 New Revision: 1485612 URL: http://svn.apache.org/r1485612 Log: Be smarter about when we remove a socket from the Poller. Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net

svn commit: r1485611 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 08:31:04 2013 New Revision: 1485611 URL: http://svn.apache.org/r1485611 Log: -1 means infinite timeout, not use socket timeout Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndp

svn commit: r1485603 - /tomcat/trunk/test/org/apache/catalina/tribes/group/interceptors/TestOrderInterceptor.java

2013-05-23 Thread markt
Author: markt Date: Thu May 23 08:03:43 2013 New Revision: 1485603 URL: http://svn.apache.org/r1485603 Log: Sometimes need a little more time for the cluster to establish Modified: tomcat/trunk/test/org/apache/catalina/tribes/group/interceptors/TestOrderInterceptor.java Modified: tomcat/tr