Send a 400 response back to the client if there is a mapping exception #134

Merged
merged 1 commit into from Mar 2, 2013

3 participants

@jsanda
Collaborator

This is some initial error handling support which I am sure is likely to change
one all of the operations are ported to handlers. Prior to this though we were
simply swallowing the exception. The client would hang indefinitely waiting for
a response, making debugging difficult. With this we are propagating the stack
trace back to the client which should really help with debugging errors.

@jsanda jsanda Send a 400 response back to the client if there is a mapping exception
This is some initial error handling support which I am sure is likely to change
one all of the operations are ported to handlers. Prior to this though we were
simply swallowing the exception. The client would hang indefinitely waiting for
a response, making debugging difficult. With this we are propagating the stack
trace back to the client which should really help with debugging errors.
27b3f36
@zznate zznate merged commit 41c2f72 into zznate:master Mar 2, 2013
@edwardcapriolo
Collaborator

Remember we have tow endpoints json and json smile. Ideally these should do the least work possible, although this change is fine. But from now on they must both do the same thing. We cant update one and notthe other.

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