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

Remove headers that do not affect CORS decision-making from request adapter logging output #9176

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

tjquinno
Copy link
Member

Description

Resolves #9171

CORS processing uses only a few headers to make its decisions. This PR streamlines which headers are logged (at FINE level) to track the CORS decisions.

Documentation

No doc impact.

@tjquinno tjquinno requested review from spericas and barchetta August 20, 2024 22:50
@tjquinno tjquinno self-assigned this Aug 20, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 20, 2024
@tjquinno tjquinno merged commit e4a9248 into helidon-io:helidon-3.x Aug 21, 2024
12 checks passed
@tjquinno tjquinno deleted the 3.x-cors-headers branch August 21, 2024 00:13
@tjquinno tjquinno linked an issue Aug 21, 2024 that may be closed by this pull request
@barchetta barchetta added the 3.x Issues for 3.x version branch label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Issues for 3.x version branch OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[3.x] Streamline CORS RequestAdapter toString method
2 participants