You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2015/11/06 10:33:34 UTC

svn commit: r1712925 - /tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java

Author: remm
Date: Fri Nov  6 09:33:34 2015
New Revision: 1712925

URL: http://svn.apache.org/viewvc?rev=1712925&view=rev
Log:
Follow up to r1712529: AJP should return false to comet rather than throwing an unsupported exception.

Modified:
    tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java

Modified: tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java?rev=1712925&r1=1712924&r2=1712925&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java (original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java Fri Nov  6 09:33:34 2015
@@ -619,8 +619,9 @@ public abstract class AbstractAjpProcess
         }
         case IS_COMET: {
             // HTTP connections only. Unsupported for AJP.
-            throw new UnsupportedOperationException(
-                    sm.getString("ajpprocessor.comet.notsupported"));
+            AtomicBoolean result = (AtomicBoolean) param;
+            result.set(false);
+            break;
         }
         case AVAILABLE: {
             if (available()) {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r1712925 - /tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java

Posted by Mark Thomas <ma...@apache.org>.
On 06/11/2015 09:33, remm@apache.org wrote:
> Author: remm
> Date: Fri Nov  6 09:33:34 2015
> New Revision: 1712925
> 
> URL: http://svn.apache.org/viewvc?rev=1712925&view=rev
> Log:
> Follow up to r1712529: AJP should return false to comet rather than throwing an unsupported exception.

Tx.

Mark

> 
> Modified:
>     tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java
> 
> Modified: tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java
> URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java?rev=1712925&r1=1712924&r2=1712925&view=diff
> ==============================================================================
> --- tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java (original)
> +++ tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AbstractAjpProcessor.java Fri Nov  6 09:33:34 2015
> @@ -619,8 +619,9 @@ public abstract class AbstractAjpProcess
>          }
>          case IS_COMET: {
>              // HTTP connections only. Unsupported for AJP.
> -            throw new UnsupportedOperationException(
> -                    sm.getString("ajpprocessor.comet.notsupported"));
> +            AtomicBoolean result = (AtomicBoolean) param;
> +            result.set(false);
> +            break;
>          }
>          case AVAILABLE: {
>              if (available()) {
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org