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

Security headers #1461

Merged
merged 10 commits into from May 1, 2014
Merged

Security headers #1461

merged 10 commits into from May 1, 2014

Conversation

@nlf
Copy link
Member

@nlf nlf commented Feb 26, 2014

Some options to include various basic security headers

@hueniverse
Copy link
Contributor

@hueniverse hueniverse commented Feb 26, 2014

I'm trying to decide if this should be on by default.

@nlf
Copy link
Member Author

@nlf nlf commented Feb 26, 2014

Yeah, I wasn't sure so I left it disabled by default

@dacbd
Copy link
Contributor

@dacbd dacbd commented Apr 23, 2014

+1 merge please, this would be extremely helpful

@phated
Copy link

@phated phated commented Apr 24, 2014

I'd like to see these also. Take down the kraken ;)

xss: Joi.boolean(),
noOpen: Joi.boolean(),
noSniff: Joi.boolean()
}).allow(null, false, true),
Copy link
Contributor

@hueniverse hueniverse May 1, 2014

Reuse the block above

@hueniverse hueniverse added this to the 4.1.0 milestone May 1, 2014
@hueniverse hueniverse self-assigned this May 1, 2014
hueniverse pushed a commit that referenced this issue May 1, 2014
@hueniverse hueniverse merged commit 9045b3d into hapijs:master May 1, 2014
1 check was pending
@nlf nlf deleted the security branch May 1, 2014
@lock lock bot locked as resolved and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants