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

uhv: http-parser: add option to skip header validation #19750

Closed
Tracked by #20261
ameily opened this issue Jan 31, 2022 · 1 comment · Fixed by #20872
Closed
Tracked by #20261

uhv: http-parser: add option to skip header validation #19750

ameily opened this issue Jan 31, 2022 · 1 comment · Fixed by #20872
Assignees
Labels
area/http enhancement Feature requests. Not bugs or questions. no stalebot Disables stalebot from closing an issue

Comments

@ameily
Copy link
Contributor

ameily commented Jan 31, 2022

As part of the unified header validation component, the http-parser library will be modified to add an option to disable header validation on a per-request basis. The intent is that header validation will be performed by the UHV and not by each codec / library.

http-parser currently controls how header validation is performed on the compile-time HTTP_PARSER_STRICT preprocessor definition, which is enabled by default. This issue will change http-parser to make this configurable at runtime via a new option.

This is blocked until #19749 is completed.

@github-actions
Copy link

github-actions bot commented Mar 6, 2022

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Mar 6, 2022
@yanavlasov yanavlasov added no stalebot Disables stalebot from closing an issue area/http and removed stale stalebot believes this issue/PR has not been touched recently labels Mar 8, 2022
@yanavlasov yanavlasov self-assigned this Mar 8, 2022
ameily added a commit to trail-of-forks/envoy that referenced this issue Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/http enhancement Feature requests. Not bugs or questions. no stalebot Disables stalebot from closing an issue
Projects
None yet
3 participants