Skip to content

Errors when preparing a response now emit internalError correctly - #716

Merged
hueniverse merged 5 commits into
hapijs:masterfrom
geek:feature/tests
Mar 29, 2013
Merged

hueniverse merged 5 commits into
hapijs:masterfrom
geek:feature/tests

Conversation

@geek

@geek geek commented Mar 27, 2013

Copy link
Copy Markdown
Member

Closes #713

Comment thread lib/response/index.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given all the other cases where response is locally replaced, this should move right before we call callback.

hueniverse pushed a commit that referenced this pull request Mar 29, 2013
Errors when preparing a response now emit internalError correctly
@hueniverse
hueniverse merged commit 21ff890 into hapijs:master Mar 29, 2013
jmonster pushed a commit to jmonster/hapi that referenced this pull request Feb 10, 2014
Errors when preparing a response now emit internalError correctly
@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.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

internalError event not emitted on view error.

2 participants