Skip to content

Commit

Permalink
[WFLY-3471] JMSContext injection
Browse files Browse the repository at this point in the history
* fix JMSContext injection to use the correct CDI scope (request or
  transaction) depending on the presence of a transaction.
* Inject @RequestScoped and @TransactionScoped beans to let CDI handle
  the bean lifecycle.
* move injection code in the
  org.wildfly.extension.messaging.activemq.deployment.injection package
* add test asserting that a @RequestScoped injected JMSContext is
  properly cleaned up at the end of the request

JIRA: https://issues.jboss.org/browse/WFLY-3471
  • Loading branch information
jmesnil committed Jun 6, 2016
1 parent b812136 commit 193ee4d
Show file tree
Hide file tree
Showing 15 changed files with 995 additions and 511 deletions.
Expand Up @@ -67,6 +67,7 @@
<module name="org.jboss.as.connector" />
<module name="org.jboss.as.security"/>
<module name="org.jboss.vfs"/>
<module name="org.jboss.jts"/>
<module name="org.picketbox"/>
<module name="org.jboss.jboss-transaction-spi"/>
<module name="org.jboss.remote-naming"/>
Expand Down
Expand Up @@ -29,7 +29,6 @@
import org.jboss.as.server.DeploymentProcessorTarget;
import org.jboss.as.server.deployment.Phase;
import org.jboss.dmr.ModelNode;
import org.wildfly.extension.messaging.activemq.deployment.CDIDeploymentProcessor;
import org.wildfly.extension.messaging.activemq.deployment.DefaultJMSConnectionFactoryBindingProcessor;
import org.wildfly.extension.messaging.activemq.deployment.DefaultJMSConnectionFactoryResourceReferenceProcessor;
import org.wildfly.extension.messaging.activemq.deployment.JMSConnectionFactoryDefinitionAnnotationProcessor;
Expand All @@ -39,6 +38,7 @@
import org.wildfly.extension.messaging.activemq.deployment.MessagingDependencyProcessor;
import org.wildfly.extension.messaging.activemq.deployment.MessagingXmlInstallDeploymentUnitProcessor;
import org.wildfly.extension.messaging.activemq.deployment.MessagingXmlParsingDeploymentUnitProcessor;
import org.wildfly.extension.messaging.activemq.deployment.injection.CDIDeploymentProcessor;

/**
* Add handler for the messaging subsystem.
Expand Down

0 comments on commit 193ee4d

Please sign in to comment.