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

Add support for composite AuthN and AuthZ filters #1488

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Add support for composite AuthN and AuthZ filters #1488

merged 1 commit into from
Apr 3, 2024

Conversation

tylerwowen
Copy link
Contributor

@tylerwowen tylerwowen commented Mar 2, 2024 •
edited
Loading

With CompositeAuthenticationFactory and CompositeAuthorizationFactory, Teletraan can be configured to run with Envoy authenticator and Pastis authorizer.

Tests

Validated the change will not affect normal operation without changing configurations in dev1.

Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

@tylerwowen tylerwowen requested a review from a team as a code owner March 2, 2024 00:26
@tylerwowen tylerwowen mentioned this pull request Mar 2, 2024
Merged
@github-actions github-actions bot added the deploy-service Includes changes to deploy-service label Mar 2, 2024
@tylerwowen tylerwowen removed the request for review from a team March 2, 2024 00:27
@tylerwowen tylerwowen marked this pull request as draft March 2, 2024 00:27
@tylerwowen tylerwowen added this to the Teletraan AuthN & AuthZ milestone Mar 2, 2024
@tylerwowen tylerwowen mentioned this pull request Mar 4, 2024
Merged
@tylerwowen tylerwowen force-pushed the spr/master/bf5d6dfa branch 2 times, most recently from b3be719 to 86eaa28 Compare March 5, 2024 03:23
@tylerwowen tylerwowen marked this pull request as ready for review March 5, 2024 03:25
@tylerwowen tylerwowen force-pushed the spr/master/bf5d6dfa branch 2 times, most recently from 7f2b3d2 to 91aaf55 Compare March 6, 2024 19:01
@tylerwowen tylerwowen force-pushed the spr/master/eb5c1322 branch 2 times, most recently from 577124d to c38962a Compare March 9, 2024 00:32
@tylerwowen tylerwowen force-pushed the spr/master/bf5d6dfa branch 2 times, most recently from 84e38e3 to 2fc4330 Compare March 12, 2024 19:01
@tylerwowen tylerwowen force-pushed the spr/master/eb5c1322 branch 2 times, most recently from f7d0472 to 84c1ac7 Compare March 26, 2024 01:07
@tylerwowen tylerwowen force-pushed the spr/master/bf5d6dfa branch 2 times, most recently from 163bd3c to 52068ea Compare March 26, 2024 16:54
@tylerwowen tylerwowen force-pushed the spr/master/eb5c1322 branch 2 times, most recently from 2532338 to 36e82b5 Compare March 27, 2024 00:33
@tylerwowen tylerwowen force-pushed the spr/master/bf5d6dfa branch 2 times, most recently from 5b174d7 to 0f4c1ee Compare March 27, 2024 21:12
@tylerwowen tylerwowen force-pushed the spr/master/eb5c1322 branch 2 times, most recently from 860538d to fc9919c Compare March 28, 2024 00:07
Base automatically changed from spr/master/bf5d6dfa to master March 28, 2024 22:22
@tylerwowen tylerwowen dismissed ntascii’s stale review March 28, 2024 22:22

The base branch was changed.

@tylerwowen tylerwowen requested a review from ntascii April 3, 2024 00:00
@tylerwowen tylerwowen merged commit c6003de into master Apr 3, 2024
5 checks passed
@tylerwowen tylerwowen deleted the spr/master/eb5c1322 branch April 3, 2024 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ntascii ntascii ntascii approved these changes

Assignees
No one assigned
Labels
deploy-service Includes changes to deploy-service
Projects
None yet
Milestone
Teletraan AuthN & AuthZ
Development

Successfully merging this pull request may close these issues.
2 participants
@tylerwowen @ntascii