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

envoy: separate authenticate filter chain #4478

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kenjenkins
Copy link
Contributor

Summary

Add support for configuring a separate filter chain for the authenticate service. This allows us to avoid prompting for a client certificate twice during the login flow (once for the route domain and a second time for the authenticate domain).

Related issues

#4364

User Explanation

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

Add support for configuring a separate filter chain for the authenticate
service. This allows us to avoid prompting for a client certificate
twice during the login flow (once for the route domain and a second time
for the authenticate domain).
@coveralls
Copy link

coveralls commented Aug 21, 2023

Coverage Status

coverage: 64.296% (+0.3%) from 64.033% when pulling 94ee93a on kenjenkins/separate-authenticate-filter-chain into 379abec on main.

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.

None yet

2 participants