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

Use Pastis for all user authorizations. #1629

Merged

tylerwowen merged 2 commits into master from touyang/pastis_for_all_users

May 28, 2024

Contributor

tylerwowen commented May 25, 2024

When CompositeAuthorizationFactory is used, only return Pastis authorizer for UserPrincipal.
Only create 1 BasePastisAuthorizer instance.
Change the order of authenticators to favor envoyAuthFilter.

Tests

Added unit test cases.


          Use Pastis for all user authorizations.

9f227de

tylerwowen requested a review from a team as a code owner

May 25, 2024 00:19

github-actions bot added the deploy-service label


          exclude tests

0e8ea01

osoriano approved these changes

View reviewed changes

tylerwowen merged commit 66edfd7 into master

5 checks passed

tylerwowen deleted the touyang/pastis_for_all_users branch

May 28, 2024 18:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels