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

Add an option to specify the CSRF header name #103

Merged
merged 3 commits into from Apr 23, 2018

Conversation

@hollandben
Copy link
Contributor

hollandben commented Sep 21, 2017

No description provided.

@jonathansamines

This comment has been minimized.

Copy link
Contributor

jonathansamines commented Dec 20, 2017

Hi @hollandben This change looks great. However there are some conflicts caused by the latests upgrade to support Hapi 17. Do you have some time to resolve them? I would like merge this.

As a side note, what do you think of adding an additional test-case for "custom header name" in addition to the existing one with the default value?

@hollandben

This comment has been minimized.

Copy link
Contributor Author

hollandben commented Dec 21, 2017

Sure, will get it done before the end of the week

@hollandben hollandben force-pushed the hollandben:add-support-for-custom-header branch 2 times, most recently from fb4df7b to c4429a6 Dec 21, 2017
@hollandben hollandben force-pushed the hollandben:add-support-for-custom-header branch from c4429a6 to 8fbdabf Dec 21, 2017
geek added 2 commits Apr 23, 2018
@geek geek added this to the 7.1.0 milestone Apr 23, 2018
@geek geek self-assigned this Apr 23, 2018
@geek geek merged commit 98a1d50 into hapijs:master Apr 23, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.