Hello Greg Wilkins thanks for bringing this to our attention. First of all, Weld 2.4 is no longer actively developed (that was CDI 1.2 impl) so we should be talking about Weld 3.x (CDI 2.0 impl) integration here. I will change the bits in issue description. The current integration, as is, predates my involvement with the project, so I'll have to take a closer look at how and why are things done so please bear with me I've quick-scanned the discussion(s) you linked as well. If I understand it correctly (please correct me here):
- the problem with Jetty 10 is ATM that Weld uses a removed class (org.eclipse.jetty.servlet.ServletContextHandler$Decorator)
- quick fix would be to simply replace it with org.eclipse.jetty.util.Decorator which existed for Jetty 9.4 and is still there for Jetty 10
- the proposed solution is to avoid these hard dependencies and instead go via context attributes
- how will you make sure the classes added this way are actually sensible Decorators without depending on(implementing) the API class?
|