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 onRequest to request options #193

Merged
merged 1 commit into from Sep 1, 2017
Merged

Add onRequest to request options #193

merged 1 commit into from Sep 1, 2017

Conversation

@geek
Copy link
Member

geek commented Sep 1, 2017

No description provided.

@geek geek requested a review from kanongil Sep 1, 2017
Copy link
Member

kanongil left a comment

It needs to also call onRequest for the return req on line 296.

Otherwise, LGTM.

@geek geek force-pushed the geek:feature branch from 4db28d7 to dfc7fa8 Sep 1, 2017
@geek geek self-assigned this Sep 1, 2017
@geek geek added the feature label Sep 1, 2017
@geek geek added this to the 12.4.0 milestone Sep 1, 2017
@geek geek merged commit 3e6fd94 into hapijs:master Sep 1, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@geek geek deleted the geek:feature branch Sep 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.