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

Added validation for request headers. #1589

Merged
merged 1 commit into from May 18, 2014
Merged

Added validation for request headers. #1589

merged 1 commit into from May 18, 2014

Conversation

@cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Apr 24, 2014

At least partially addresses #1588.

When will object.unknown() from joi land in hapi? Once it does, it should be applied to header validation due to all the extraneous headers sent with most requests. Or is there a better way to do it today?

@hueniverse
Copy link
Contributor

@hueniverse hueniverse commented Apr 24, 2014

I am going to solve it a different way in v5 by moving the rules to the validate level instead of the three options today.

Loading

@hueniverse hueniverse closed this Apr 24, 2014
@hueniverse hueniverse reopened this May 18, 2014
@hueniverse hueniverse added this to the 5.0.0 milestone May 18, 2014
@hueniverse hueniverse self-assigned this May 18, 2014
@hueniverse hueniverse merged commit 2d40337 into hapijs:master May 18, 2014
1 check passed
Loading
@cjihrig cjihrig deleted the header-validation branch Jun 29, 2014
@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants