On 06/01/2017 20:22, Violeta Georgieva wrote: > 2017-01-06 21:29 GMT+02:00 Mark Thomas <ma...@apache.org>: >> >> On 06/01/2017 15:57, Mark Thomas wrote: >>> On 06/01/2017 14:45, ma...@apache.org wrote: >>>> Author: markt >>>> Date: Fri Jan 6 14:45:03 2017 >>>> New Revision: 1777619 >>>> >>>> URL: http://svn.apache.org/viewvc?rev=1777619&view=rev >>>> Log: >>>> Partial revert of r1534619. >>>> The caller has to be able to differentiate between no data and EOF. >>> >>> While this fixed the looping issue, it appears to have broken multiple >>> Autobahn tests. I'm working on a better fix. >> >> I'm not sure if this is good news or bad news. The Autobahn test >> failures persist after I revet this fix. >> >> Digging... > > Which test is the failing one? Connector type - NIO/NIO2/APR? > I ran it on my machine - MAC OS, Connector - NIO and I do not see failures.
Fairly old autobahn tests: (I haven't updated them in a while) User-Agent: AutobahnTestSuite/0.7.5-0.10.1 APR/native Windows Not exactly sure what is going wrong yet. Mark > >> Mark >> >> >>> >>> Mark >>> >>> >>>> >>>> Modified: >>>> > tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFrameServer.java >>>> tomcat/trunk/webapps/docs/changelog.xml >>>> >>>> Modified: > tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFrameServer.java >>>> URL: > http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFrameServer.java?rev=1777619&r1=1777618&r2=1777619&view=diff >>>> > ============================================================================== >>>> --- > tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFrameServer.java > (original) >>>> +++ > tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFrameServer.java Fri > Jan 6 14:45:03 2017 >>>> @@ -16,6 +16,7 @@ >>>> */ >>>> package org.apache.tomcat.websocket.server; >>>> >>>> +import java.io.EOFException; >>>> import java.io.IOException; >>>> import java.nio.ByteBuffer; >>>> >>>> @@ -61,7 +62,9 @@ public class WsFrameServer extends WsFra >>>> > inputBuffer.position(inputBuffer.limit()).limit(inputBuffer.capacity()); >>>> int read = socketWrapper.read(false, inputBuffer); >>>> inputBuffer.limit(inputBuffer.position()).reset(); >>>> - if (read <= 0) { >>>> + if (read < 0) { >>>> + throw new EOFException(); >>>> + } else if (read == 0) { >>>> return; >>>> } >>>> if (log.isDebugEnabled()) { >>>> >>>> Modified: tomcat/trunk/webapps/docs/changelog.xml >>>> URL: > http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1777619&r1=1777618&r2=1777619&view=diff >>>> > ============================================================================== >>>> --- tomcat/trunk/webapps/docs/changelog.xml (original) >>>> +++ tomcat/trunk/webapps/docs/changelog.xml Fri Jan 6 14:45:03 2017 >>>> @@ -53,6 +53,14 @@ >>>> </fix> >>>> </changelog> >>>> </subsection> >>>> + <subsection name="WebSocket"> >>>> + <changelog> >>>> + <fix> >>>> + Prevent potential processing loop on unexpected WebSocket > connection >>>> + closure. (markt) >>>> + </fix> >>>> + </changelog> >>>> + </subsection> >>>> <subsection name="Other"> >>>> <changelog> >>>> <update> >>>> >>>> >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >>>> For additional commands, e-mail: dev-h...@tomcat.apache.org >>>> >>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >>> For additional commands, e-mail: dev-h...@tomcat.apache.org >>> >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >> For additional commands, e-mail: dev-h...@tomcat.apache.org >> > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org