/juju

resource/api: always send a response to the client #4596

Merged
merged 1 commit into from Mar 3, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@davecheney @jujubot
@davecheney
Contributor

davecheney commented Mar 2, 2016

Always send a response to the client, even when an error occurs.

(Review request: http://reviews.vapour.ws/r/4036/)
@davecheney
resource/api: always send a response to the client 
Always send a response to the client, even when an error occurs.
fc99c1b
@davecheney
Contributor

davecheney commented Mar 2, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Mar 2, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented Mar 2, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/6644
@davecheney
Contributor

davecheney commented Mar 3, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Mar 3, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Mar 3, 2016

@jujubot
Merge pull request #4596 from davecheney/resource-api-always-send-res… 
…ponse-even-during-error

resource/api: always send a response to the client

Always send a response to the client, even when an error occurs.

(Review request: http://reviews.vapour.ws/r/4036/)
939cbbf

@jujubot jujubot merged commit 939cbbf into juju:master Mar 3, 2016

@davecheney davecheney deleted the davecheney:resource-api-always-send-response-even-during-error branch Mar 3, 2016

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