Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ARTEMIS-4467 Core client code visibility change required. 

* Since the constructors in RemotingConnectionImpl can be used from the server and the client.
If the server calling code is in a different classloader then the
constructor can't be called.
* same for the 'active' boolean property of ActiveMQChannelHandler

Issue: https://issues.apache.org/jira/browse/ARTEMIS-4467

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
This commit is contained in:
Emmanuel Hugonnet 2023-10-23 17:15:50 +02:00 committed by clebertsuconic
parent 6649ecf963
commit 37273701ea
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/RemotingConnectionImpl.java
View File

@ -96,7 +96,7 @@ public class RemotingConnectionImpl extends AbstractRemotingConnection implement
/*
* Create a server side connection
*/
RemotingConnectionImpl(final PacketDecoder packetDecoder,
public RemotingConnectionImpl(final PacketDecoder packetDecoder,
final Connection transportConnection,
final List<Interceptor> incomingInterceptors,
final List<Interceptor> outgoingInterceptors,

2
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/ActiveMQChannelHandler.java
View File

@ -40,7 +40,7 @@ public class ActiveMQChannelHandler extends ChannelDuplexHandler {
private final BaseConnectionLifeCycleListener<?> listener;
volatile boolean active;
protected volatile boolean active;
private final Executor listenerExecutor;