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

Fix docs w.r.t. helmet.csp #20

Merged
merged 1 commit into from Nov 4, 2016

Conversation

Projects
None yet
2 participants
@jasonkarns
Contributor

jasonkarns commented Nov 4, 2016

helmet.csp is not defined. It should be helmet.contentSecurityPolicy

see also: https://github.com/helmetjs/helmet/blob/master/index.js#L32

Fix docs w.r.t. helmet.csp
`helmet.csp` is not defined. It should be `helmet.contentSecurityPolicy`

see also: https://github.com/helmetjs/helmet/blob/master/index.js#L32

@EvanHahn EvanHahn merged commit e8f6c70 into helmetjs:master Nov 4, 2016

@EvanHahn

This comment has been minimized.

Show comment
Hide comment
@EvanHahn

EvanHahn Nov 4, 2016

Member

Good catch! Thanks.

Can I add you to the contributors list? If so, give me a name and URL to use!

Member

EvanHahn commented Nov 4, 2016

Good catch! Thanks.

Can I add you to the contributors list? If so, give me a name and URL to use!

@jasonkarns jasonkarns deleted the jasonkarns:patch-1 branch Nov 4, 2016

@jasonkarns

This comment has been minimized.

Show comment
Hide comment
@jasonkarns

jasonkarns Nov 4, 2016

Contributor

Sure. Jason Karns jasonkarns.com

Contributor

jasonkarns commented Nov 4, 2016

Sure. Jason Karns jasonkarns.com

EvanHahn added a commit that referenced this pull request Nov 6, 2016

@EvanHahn

This comment has been minimized.

Show comment
Hide comment
@EvanHahn

EvanHahn Nov 6, 2016

Member
Member

EvanHahn commented Nov 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment