Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

It does not take effect when acl is enabled in the proxy #6185

Closed
Oliverwqcwrw opened this issue Feb 26, 2023 · 3 comments · Fixed by #6186
Closed

It does not take effect when acl is enabled in the proxy #6185

Oliverwqcwrw opened this issue Feb 26, 2023 · 3 comments · Fixed by #6186
Labels
module/acl module/proxy type/bug

Comments

@Oliverwqcwrw
Copy link
Member

BUG REPORT

  1. Please describe the issue you observed:
  • What did you do (The steps to reproduce)?

image

It has no spi file

  • What is expected to see?

  • What did you see instead?

  1. Please tell us about your environment:

  2. Other information (e.g. detailed explanation, logs, related issues, suggestions on how to fix, etc):
Oliverwqcwrw added a commit to Oliverwqcwrw/rocketmq that referenced this issue Feb 26, 2023
@Oliverwqcwrw
[ISSUE apache#6185]Fix It does not take effect when acl is enabled in…
94f7e91 
… the proxy
@Oliverwqcwrw Oliverwqcwrw mentioned this issue Feb 26, 2023
Merged
6 tasks
@drpmma
Copy link
Contributor

drpmma commented Feb 27, 2023

It appears that the commit 1116de4 (ISSUE #5295) changed the behavior of the ServiceProvider in a way that caused this issue.

@drpmma
Copy link
Contributor

drpmma commented Feb 27, 2023 •
edited

It is possible that the merging issue caused the problem, and the service loader in proxy acl is not taken into consideration.

Oliverwqcwrw added a commit to Oliverwqcwrw/rocketmq that referenced this issue Feb 27, 2023
@Oliverwqcwrw
[ISSUE apache#6185]Fix It does not take effect when acl is enabled in…
d6c2305 
… the proxy
@Oliverwqcwrw
Copy link
Member Author

It appears that the commit 1116de4 (ISSUE #5295) changed the behavior of the ServiceProvider in a way that caused this issue.

It seems to have this problem all the time.

drpmma pushed a commit that referenced this issue Feb 28, 2023
@Oliverwqcwrw @drpmma
[ISSUE #6185]Fix It does not take effect when acl is enabled in the p…
f712bc8 
…roxy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
module/acl module/proxy type/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #6185] Fix It does not take effect when acl is enabled in the proxy Oliverwqcwrw/rocketmq
2 participants
@drpmma @Oliverwqcwrw