Le jeu. 22 avr. 2021 à 17:32, Mark Thomas <[email protected]> a écrit :
> On 22/04/2021 16:24, Romain Manni-Bucau wrote: > > Hi, > > > > Is it possible to reuse/add/have a SPI to create ws coders/decoders? > > > > Currently it is a hardcoded factory doing a "new" > > (org.apache.tomcat.websocket.Util#getDecoders for decoders, for encoders > it > > is a bit worse since there is an instance as a check here > > org.apache.tomcat.websocket.server.WsServerContainer#validateEncoders and > > the instantiation happens here > > org.apache.tomcat.websocket.WsRemoteEndpointImplBase#setEncoders). > > > > Best would likely be to reuse tomcat instance manager since it would make > > it working OOTB for integrations/users and also enable to have a proper > > lifecycle management (destroyInstance). > > > > Wdyt? > > -> https://github.com/eclipse-ee4j/websocket-api/issues It is already required to cover 7.1.1. Currently tomee worked around it by dropping default sci and having a custom one override what it can but it is not generally reusable in cdi app easily so spi is needed and destroy phase too by spec ;). > > Mark > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
