Skip to content

Error standardization #1144

@mattdailis

Description

@mattdailis

Across the board, we would like Aerie to return 200 status codes for all requests that don't result in an internal server error.

We want to make sure that the responses are formatted in a way that Hasura understands - we want to eliminate the "message" field was not present error message.

Follow up work would be to make sure that we do not throw internal server errors for situations that a user can fix. This means taking a hard look at all places where we throw new Error(...). A possible outcome might be to replace usages of Error with a subtype of RuntimeException, or (shudder) catch Error in the merlin bindings, which would allow us to cleanly report the error.

Metadata

Metadata

Assignees

Labels

investigateIssue needs further investigationrefactorA code change that neither fixes a bug nor adds a feature

Type

No type

Projects

Status

Todo

Relationships

None yet

Development

No branches or pull requests

Issue actions