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 headers control to dex web server #3339

Merged
merged 1 commit into from Mar 11, 2024

Conversation

nabokihms
Copy link
Member

Overview

Customization of headers in the authentication server is crucial for enforcing stringent security measures by allowing the inclusion of specific headers required for authentication protocols and compliance standards. This customization ensures that authentication requests are processed securely, mitigating potential vulnerabilities and ensuring adherence to security policies.

What this PR does / why we need it

Related #3320

Special notes for your reviewer

Customization of headers in the authentication server is crucial for enforcing stringent security measures by allowing the inclusion of specific headers required for authentication protocols and compliance standards. This customization ensures that authentication requests are processed securely, mitigating potential vulnerabilities and ensuring adherence to security policies.

Signed-off-by: m.nabokikh <maksim.nabokikh@flant.com>
@hopeshared-lee
Copy link

The change looks good to me. Thanks! @nabokihms

@nabokihms
Copy link
Member Author

It seems that #2266 is superseded by this PR

@nabokihms nabokihms merged commit 088339f into dexidp:master Mar 11, 2024
9 of 10 checks passed
@nabokihms nabokihms deleted the dex-headers-control branch March 11, 2024 21:48
@hopeshared-lee
Copy link

It's a great news to have the pr delivered!
Do you have a planned release date? Thanks!

@nabokihms
Copy link
Member Author

@hopeshared-lee the release of Dex v2.39.0 is planned for the last week of March.

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

Successfully merging this pull request may close these issues.

None yet

3 participants