This is an automated email from the ASF dual-hosted git repository.
jinrongtong pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/rocketmq.git
The following commit(s) were added to refs/heads/develop by this push:
new ea1228a30a [ISSUE #8966] Feat: add remote address information to acl
perm error
ea1228a30a is described below
commit ea1228a30ad4d12259a653585eb73ec12e3374b1
Author: Humkum <[email protected]>
AuthorDate: Mon Dec 2 10:04:17 2024 +0800
[ISSUE #8966] Feat: add remote address information to acl perm error
---
.../org/apache/rocketmq/remoting/netty/NettyRemotingAbstract.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git
a/remoting/src/main/java/org/apache/rocketmq/remoting/netty/NettyRemotingAbstract.java
b/remoting/src/main/java/org/apache/rocketmq/remoting/netty/NettyRemotingAbstract.java
index b0c7099b9d..3d4e62f943 100644
---
a/remoting/src/main/java/org/apache/rocketmq/remoting/netty/NettyRemotingAbstract.java
+++
b/remoting/src/main/java/org/apache/rocketmq/remoting/netty/NettyRemotingAbstract.java
@@ -320,9 +320,10 @@ public abstract class NettyRemotingAbstract {
return () -> {
Exception exception = null;
RemotingCommand response;
+ String remoteAddr = null;
try {
- String remoteAddr =
RemotingHelper.parseChannelRemoteAddr(ctx.channel());
+ remoteAddr =
RemotingHelper.parseChannelRemoteAddr(ctx.channel());
try {
doBeforeRpcHooks(remoteAddr, cmd);
} catch (AbortProcessException e) {
@@ -359,7 +360,7 @@ public abstract class NettyRemotingAbstract {
response.setOpaque(opaque);
writeResponse(ctx.channel(), cmd, response);
} catch (Throwable e) {
- log.error("process request exception", e);
+ log.error("process request exception, remoteAddr: {}",
remoteAddr, e);
log.error(cmd.toString());
if (!cmd.isOnewayRPC()) {