The GitHub Actions job "CI" on 
rocketmq-dashboard.git/fix/yy-acl-tencent-role-id has failed.
Run started by GitHub user yyqdbngt (triggered by yyqdbngt).

Head commit for run:
3dfb7f8e87e241082558d5ff5a90ef26290d70e9 / yyqdbngt <[email protected]>
fix(ai): accept a Tencent role name as the ACL rule identifier

AclUpdateToolHandler.parseId converted every rmq.acl.update identifier into a
Long and answered a non-numeric one with ACL_ID_INVALID. A Tencent ACL rule is
identified by its role name, which TencentAclService reads from the rule
principal, so the handler refused in front of the provider-aware
AclService.updateRule that supports exactly that identifier: the MCP write path
could not reach Tencent rules that the REST path can.

Return null for an identifier that is not numeric and let AclService route the
rule to the provider. On the Apache path AclService still rejects a rule without
a numeric id, and a missing identifier is still refused by the handler.

Test: AclUpdateToolHandlerTest covers the role-name identifier reaching the
service with no synthesised numeric id, and that a blank id never reaches it.

Report URL: 
https://github.com/apache/rocketmq-dashboard/actions/runs/36962905174

With regards,
GitHub Actions via GitBox

Reply via email to