cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CXF-7571) Revamp of the CXF injection implementation
Date Sat, 30 Dec 2017 15:58:00 GMT

    [ https://issues.apache.org/jira/browse/CXF-7571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16306856#comment-16306856
] 

ASF GitHub Bot commented on CXF-7571:
-------------------------------------

johnament commented on a change in pull request #351: [CXF-7571] Adding support for CDI injection
of @Context objects.
URL: https://github.com/apache/cxf/pull/351#discussion_r159124524
 
 

 ##########
 File path: systests/cdi/base/src/main/java/org/apache/cxf/systests/cdi/base/BookStoreVersion.java
 ##########
 @@ -22,20 +22,25 @@
 import javax.enterprise.context.RequestScoped;
 import javax.inject.Inject;
 import javax.ws.rs.GET;
-import javax.ws.rs.core.Context;
 import javax.ws.rs.core.HttpHeaders;
 import javax.ws.rs.core.MediaType;
 import javax.ws.rs.core.Response;
 
+import org.apache.cxf.systests.cdi.base.context.CustomContext;
+
 @RequestScoped
 public class BookStoreVersion {
     @Inject
     private String version;
-    @Context
+    @Inject
     private HttpHeaders httpHeaders;
+    @Inject
 
 Review comment:
   @reta yes, that's what I'm expecting to happen, however I'm never seeing the bean invoked
for this context object.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


>  Revamp of the CXF injection implementation
> -------------------------------------------
>
>                 Key: CXF-7571
>                 URL: https://issues.apache.org/jira/browse/CXF-7571
>             Project: CXF
>          Issue Type: Improvement
>            Reporter: Andriy Redko
>            Assignee: John D. Ament
>
> As more deep integration with CDI revealed, there are complexities in bringing together
{{@Context}}- and {{@Inject}}-based injections. Encapsulating CXF injection implementation
and than delegating the hard work to appropriate strategy (CDI, Spring, ...) would be the
right solution to address the problem at its roots.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message