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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Cannot read property 'statusCodes' of undefined #303

Merged
merged 2 commits into from Feb 17, 2015

Conversation

@AdriVanHoudt
Copy link
Contributor

AdriVanHoudt commented Feb 16, 2015

If someone can help me squashing 馃槆 (I've never done it and those commits don't show in my log 馃槓 )
Thanks to @arb for the squashing help

@AdriVanHoudt AdriVanHoudt force-pushed the AdriVanHoudt:null-response branch from acbddac to 1ad0aee Feb 17, 2015

// force response to be null to mimic client disconnect
server.on('response', function (request) {
request.response = null;

This comment has been minimized.

Copy link
@arb

arb Feb 17, 2015

Contributor

Newline after function.

@arb arb added the feature label Feb 17, 2015
@arb arb self-assigned this Feb 17, 2015
@arb arb added this to the 5.1.2 milestone Feb 17, 2015
arb added a commit that referenced this pull request Feb 17, 2015
Fix: Cannot read property 'statusCodes' of undefined. Closes #298.
@arb arb merged commit 4eb2b06 into hapijs:master Feb 17, 2015
@lloydbenson

This comment has been minimized.

Copy link
Contributor

lloydbenson commented Feb 17, 2015

Thanks for the help here! Very much appreciated. @AdriVanHoudt and @chulkilee.

@AdriVanHoudt

This comment has been minimized.

Copy link
Contributor Author

AdriVanHoudt commented Feb 17, 2015

np, glad to help!

@arb arb added bug and removed feature labels Feb 17, 2015
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鈥檛 perform that action at this time.