This is an automated email from the ASF dual-hosted git repository.

lollipop 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 797024f3f8 fix update user not valid bug (#8926)
797024f3f8 is described below

commit 797024f3f8cb51fbe6352832afd38c44f451cf38
Author: dingshuangxi888 <dingshuangxi...@gmail.com>
AuthorDate: Fri Nov 15 10:35:37 2024 +0800

    fix update user not valid bug (#8926)
---
 .../java/org/apache/rocketmq/broker/processor/AdminBrokerProcessor.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/broker/src/main/java/org/apache/rocketmq/broker/processor/AdminBrokerProcessor.java
 
b/broker/src/main/java/org/apache/rocketmq/broker/processor/AdminBrokerProcessor.java
index 381889c624..ac882e94ab 100644
--- 
a/broker/src/main/java/org/apache/rocketmq/broker/processor/AdminBrokerProcessor.java
+++ 
b/broker/src/main/java/org/apache/rocketmq/broker/processor/AdminBrokerProcessor.java
@@ -3113,7 +3113,7 @@ public class AdminBrokerProcessor implements 
NettyRequestProcessor {
                 if (old.getUserType() == UserType.SUPER && 
isNotSuperUserLogin(request)) {
                     throw new AuthenticationException("The super user can only 
be update by super user");
                 }
-                return 
this.brokerController.getAuthenticationMetadataManager().updateUser(old);
+                return 
this.brokerController.getAuthenticationMetadataManager().updateUser(user);
             }).thenAccept(nil -> response.setCode(ResponseCode.SUCCESS))
             .exceptionally(ex -> {
                 LOGGER.error("update user {} error", 
requestHeader.getUsername(), ex);

Reply via email to