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

4.1.4: URI Validation updates #9476

Open
wants to merge 1 commit into
base: helidon-4.1.x
Choose a base branch
from

Conversation

barchetta
Copy link
Member

Backport #9469 to Helidon 4.1.4

  1. Moved host validation to UriValidator, as it belongs to URI specification
  2. Updated query and fragment validation
  3. Added configuration option to disable validation if desired

* Validation of URI Query
* Moving host validation to UriValidator, as it belongs to URI concept (and is part of URI specification).
The UriValidator now has methods to validate scheme, host, query, and fragment.
Path validation is retained in `UriPath`.
Added a new configuration option to disable validation of prologue for HTTP/1.

Signed-off-by: Tomas Langer <[email protected]>
@barchetta barchetta added webserver backport Issues that are merged into a single branch, but missing in either master or previous release 4.x Version 4.x labels Nov 7, 2024
@barchetta barchetta added this to the 4.1.4 milestone Nov 7, 2024
@barchetta barchetta self-assigned this Nov 7, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Nov 7, 2024
@barchetta barchetta changed the title URI Validation updates (#9469) 4.1.4: URI Validation updates (#9469) Nov 7, 2024
@barchetta barchetta mentioned this pull request Nov 7, 2024
9 tasks
@barchetta barchetta changed the title 4.1.4: URI Validation updates (#9469) 4.1.4: URI Validation updates Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x backport Issues that are merged into a single branch, but missing in either master or previous release OCA Verified All contributors have signed the Oracle Contributor Agreement. webserver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants