You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by "Mark Thomas (Confluence)" <no...@apache.org> on 2020/06/17 17:10:05 UTC

[CONF] Apache Tomcat > WebSocket 2.0 TCK (DRAFT)

There's **1 new edit** on this page  
---  
|  
---  
|  | [![page icon](cid:page-
icon)](https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=158863846&src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db&src.mail.action=view
"page icon")  
---  
[WebSocket 2.0 TCK
(DRAFT)](https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=158863846&src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db&src.mail.action=view
"WebSocket 2.0 TCK \(DRAFT\)")  
|  |  |  |  | ![](cid:avatar_0e7b7d8d93bf536b4e2f8f7984d49115) |  | Mark
Thomas edited this page  
---  
|  
|  | Here's what changed:  
---  
|

...

  * org.apache.tomcat.websocket.DISABLE_BUILTIN_EXTENSIONS=true
  * org.apache.tomcat.websocket.ALLOW_UNSUPPORTED_EXTENSIONS=true
  * org.apache.tomcat.websocket.DEFAULT_PROCESS_PERIOD=0

Make the following changes to server.xml

  * add backgroundProcessorDelay="1" to the <Engine ... > element

## Test Suite

Download latest nightly build

https[http://download.eclipse.org/ee4j/jakartaee-
tck/](http://download.eclipse.org/ee4j/jakartaee-tck/master/nightly/websocket-
tck-2.0.0.zip)8.0.1[master/nightly/](http://download.eclipse.org/ee4j/jakartaee-
tck/master/nightly/websocket-tck-2.0.0.zip)websockettck-1.1_latest[websocket-
tck-2.0.0.zip](http://download.eclipse.org/ee4j/jakartaee-
tck/master/nightly/websocket-tck-2.0.0.zip)

Extract to WEBSOCKET_TCK_HOME

...

# Expected results (Jakarta EE 9)

A default 10.0.x build (as of 2020-05-13) running with the nightly TCK build
(as of 2020-05-13) triggers 10 test failures:

_The PRs have been applied. This TCK needs to be re-run to confirm the fixes._

2 unclear specification

  * 2 x URI <https://github.com/eclipse-ee4j/websocket-api/issues/228>   
PR to exclude <https://github.com/eclipse-ee4j/jakartaee-tck/pull/263>

...

5 Tests 'fixed' by appropriate system property configuration (see above). No
need for the server.xml changes.

  * 4 x extensions (TCK assumes invalid extensions are ignored)  
Consider making TCK smarter so it ignores other extensions the server may
advertise. Better still, detect them first and then exclude them from this
test.  
Need to discuss expected behaviour if an unknown extension is specified. Are
these required or optional?

  * 1 x timeout related test expects more frequent expiration checks

#  Expected results (Java EE 8)

A default 9.0.x build without any configuration triggers 20 test failures

2 unclear specification

  * 2 x URI [https://github.com/eclipse-ee4j/websocket-api/issues/228](https://github.com/eclipse-ee4j/websocket-api/issues/228)

11 Faulty tests

  * 1 x [https://github.com/eclipse-ee4j/jakartaee-tck/issues/33](https://github.com/eclipse-ee4j/jakartaee-tck/issues/33) (batching)
  * 2 x [https://github.com/eclipse-ee4j/jakartaee-tck/issues/35](https://github.com/eclipse-ee4j/jakartaee-tck/issues/35) (batching)
  * 8 x [https://github.com/eclipse-ee4j/jakartaee-tck/issues/37](https://github.com/eclipse-ee4j/jakartaee-tck/issues/37) (async concurrency)

5 Tests 'fixed' by appropriate system property configuration (see above)

  * 4 x extensions (TCK assumes invalid extensions are ignored)
  * 1 x timeout related test expects more frequent expiration checks

2 Tests 'fixed' by appropriate server.xml configuration

  * 2 x timeout related tests requiring even more frequent expiration checks
    * Still see intermittent failures on these

A further 19 test failures caused by Tomcat bugs have been fixed

  * 4 x Deployment failure didn't undeploy all WebSockets
  * 4 x @PathParam not validated for correct type
  * 6 x Throw DeploymentException rather than IAE
  * 1 x Improve checking of multiple @OnMessage for the same type
  * 4 x exception in encoder leading (incorrectly) to empty message

  
---  
|  |  | [Go to page
history](https://cwiki.apache.org/confluence/pages/viewpreviousversions.action?pageId=158863846&src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db "Go to page
history")  
---  
---  
| [View
page](https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=158863846&src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db&src.mail.action=view)  
---  
  
|  | [Stop watching
space](https://cwiki.apache.org/confluence/users/removespacenotification.action?spaceKey=TOMCAT&src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db&src.mail.action=stop-
watching&jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzdWIiOiJ4c3JmOjhhYTk4MDk1NjlkNDIzY2QwMTZhMDQxMzMwNmYwMGRiIiwicXNoIjoiZDAxMjE4MWU0NzE0ZWVhMDE1N2Y0M2RlNTYzMThmNjM4NjM4Y2FjYmIxM2RmMzBiYWE5MTQwYmM4YjcwYzNlNiIsImlzcyI6ImNvbmZsdWVuY2Vfbm90aWZpY2F0aW9uc0FSRUgtWFVEMS1QT1FHLUNTQU8iLCJleHAiOjE1OTMwMTg2MDUsImlhdCI6MTU5MjQxMzgwNX0.ai0rqZqVWTrvFhpepmTA-
Uf1MWSwcateKOa5VH1k78k) | •  
---|---  
[Manage
notifications](https://cwiki.apache.org/confluence/users/editmyemailsettings.action?src=mail&src.mail.product=confluence-
server&src.mail.timestamp=1592413805593&src.mail.notification=com.atlassian.confluence.plugins.confluence-
notifications-batch-plugin%3Abatching-
notification&src.mail.recipient=8aa9809569d423cd016a0413306f00db&src.mail.action=manage)  
---  
| ![Confluence logo big](cid:footer-desktop-logo)  
---  
This message was sent by Atlassian Confluence 7.5.0  
![](cid:footer-mobile-logo)  
---