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

Wrong auth provider check on broker's originalRole auth check #6873

Closed
KannarFr opened this issue May 4, 2020 · 0 comments · Fixed by #6870
Closed

Wrong auth provider check on broker's originalRole auth check #6873

KannarFr opened this issue May 4, 2020 · 0 comments · Fixed by #6870
Labels
area/proxy type/bug The PR fixed a bug or issue reported a bug

Comments

@KannarFr
Copy link
Contributor

KannarFr commented May 4, 2020

Describe the bug
If a proxy has a auth provider A, and the proxy to broker has a B auth provider usage. The broker originalAuth check uses B instead of A.
@KannarFr KannarFr added the type/bug The PR fixed a bug or issue reported a bug label May 4, 2020
@KannarFr KannarFr mentioned this issue May 4, 2020
Merged
@sijie sijie mentioned this issue May 4, 2020
Closed
sijie pushed a commit that referenced this issue May 9, 2020
@KannarFr
use originalAuthMethod on originalAuthChecker (#6870)
8381371 
Fixes #6873

Use originalAuthMethod on originalAuthProvider check.
jiazhai pushed a commit that referenced this issue May 12, 2020
@KannarFr @jiazhai
use originalAuthMethod on originalAuthChecker (#6870)
450cc15 
Fixes #6873

Use originalAuthMethod on originalAuthProvider check.
(cherry picked from commit 8381371)
Huanli-Meng pushed a commit to Huanli-Meng/pulsar that referenced this issue May 27, 2020
@KannarFr @Huanli-Meng
use originalAuthMethod on originalAuthChecker (apache#6870)
7784ebc 
Fixes apache#6873

Use originalAuthMethod on originalAuthProvider check.
addisonj pushed a commit to instructure/pulsar that referenced this issue Jun 12, 2020
@KannarFr
use originalAuthMethod on originalAuthChecker (apache#6870)
dd2b967 
Fixes apache#6873

Use originalAuthMethod on originalAuthProvider check.
(cherry picked from commit 8381371)
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this issue Aug 24, 2020
@KannarFr @huangdx0726
use originalAuthMethod on originalAuthChecker (apache#6870)
0783275 
Fixes apache#6873

Use originalAuthMethod on originalAuthProvider check.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/proxy type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

use originalAuthMethod on originalAuthChecker CleverCloud/pulsar
2 participants
@sijie @KannarFr