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.

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

Add option to verify client ID #246

Merged
merged 1 commit into from
Feb 13, 2020
Merged

Conversation

weinong
Copy link
Contributor

@weinong weinong commented Feb 7, 2020

Add an option in azure auth module to verify client ID which follows the best practice from Azure AD
It fixes #244

@weinong
Copy link
Contributor Author

weinong commented Feb 7, 2020

forced push to update doc. @karataliu PTAL

auth/providers/azure/options.go Outdated Show resolved Hide resolved
auth/providers/azure/options_test.go Outdated Show resolved Hide resolved
Signed-off-by: Weinong Wang <weinong@outlook.com>
@weinong
Copy link
Contributor Author

weinong commented Feb 11, 2020

@tamalsaha PTAL

@tamalsaha tamalsaha changed the title add option to verify client ID Add option to verify client ID Feb 13, 2020
@tamalsaha tamalsaha merged commit 9b0d71a into kubeguard:master Feb 13, 2020
@tamalsaha
Copy link
Contributor

@weinong , do you plan to open any more pr regarding this or should I cut a release?

@weinong weinong deleted the fix-244 branch February 13, 2020 07:40
@weinong
Copy link
Contributor Author

weinong commented Feb 13, 2020

@tamalsaha you can cut the release. The only thing left on my todo is to add metrics. But that will come later. Thank you!

nightfury1204 pushed a commit to nightfury1204/guard that referenced this pull request Jul 3, 2020
Add an option in azure auth module to verify client ID which follows the best practice from Azure AD
It fixes kubeguard#244 

Signed-off-by: Weinong Wang <weinong@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

azure auth provider should validate token audience
3 participants