|
Hello All,
This week I've been working on integrating Jakarta REST 5.0 CDI integration into RESTEasy. I've found a couple minor issues we'll need to address in the spec. However, I'd first like to talk about
@Context. We deprecated that annotation, but now I'm wondering if we should instead make not deprecate it and make it a CDI qualifier. The biggest reason for this would be we do need some kind of a qualifier to inject the `ServletConfig` and `ServletContext`.
If we don't have a qualifier, then there is a chance we get the wrong instance injected. Making the `@Context` a CDI qualifier would allow the implementation and user to know they're getting the correct instances injected.
The other two issues are more minor and just documentation changes I hadn't thought about until I fully implemented some things. I'll send up a PR those and for making the @Context a CDI qualifier, but that will be a separate commit in case others disagree.
It will make it easier to delete.
James R. Perkins
Software Developer
IBM
|