[
https://issues.apache.org/jira/browse/ARTEMIS-274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14969156#comment-14969156
]
ASF GitHub Bot commented on ARTEMIS-274:
----------------------------------------
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-artemis/pull/211#issuecomment-150224242
I will merge this, but i will first ammend the checkstyle issue, and the
change I asked here...
I will run the entire testsuite first.. so it will take me 1 hour before I
can merge this.
> Make ActiveMQThreadFactory instantiation running within doPrivileged block
> --------------------------------------------------------------------------
>
> Key: ARTEMIS-274
> URL: https://issues.apache.org/jira/browse/ARTEMIS-274
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Affects Versions: 1.1.0
> Reporter: Ivo Studensky
> Fix For: 1.2.0, 1.1.1
>
>
> At the moment, ActiveMQThreadFactory caches AccessControlContext inside of
> its constructor and uses it later in newThread() method invocation to create
> a thread inside of doPrivileged block. This leads to not enough permissions
> issue when a client is trying to create a new connection on a factory taken
> from JNDI, see https://issues.jboss.org/browse/WFLY-5166.
> To fix it all the invocations of the factory constructor should run within a
> privileged block to get a privileged ACC to the factory. This change should
> be similar to how it works in WildFly right now.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
