Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[ISSUE #9133] Credentials enhancement #9134

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

zeyu-zh
Copy link

@zeyu-zh zeyu-zh commented Jan 15, 2025

Which Issue(s) This PR Fixes

Fixes #9133

Brief Description

How Did You Test This Change?

Not yet

@@ -0,0 +1,14 @@
package org.apache.rocketmq.acl.common;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, a license is required here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR has been updated. Thanks for your feedback!

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 47.97%. Comparing base (14ca1ce) to head (50d63e4).
Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
...g/apache/rocketmq/acl/common/AclClientRPCHook.java 50.00% 3 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9134      +/-   ##
=============================================
+ Coverage      47.93%   47.97%   +0.03%     
- Complexity     12046    12058      +12     
=============================================
  Files           1321     1322       +1     
  Lines          93024    93032       +8     
  Branches       11926    11926              
=============================================
+ Hits           44594    44635      +41     
+ Misses         42916    42857      -59     
- Partials        5514     5540      +26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@qianye1001 qianye1001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] A more flexible way of credential configuration
4 participants