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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Errors do not respect the June2018 specs #71

Open
bboure opened this issue Dec 1, 2019 · 1 comment
Open

Errors do not respect the June2018 specs #71

bboure opened this issue Dec 1, 2019 · 1 comment

Comments

@bboure
Copy link

@bboure bboure commented Dec 1, 2019

Since the June 2008 specs, Errors adds the extensions entry. It also discourages the usage of any other additional entries.

GraphQL services should not provide any additional entries to the error format since they could conflict with additional entries that may be added in future versions of this specification.

AppSync should honor those specs and drop the data, errorInfo and errorType entries from the errors. Instead, it should implement the extensions one.

$util.error should probably become $util.error(String) and $util.error(String, Object)

Where the first argument is the message string, and the second an optional extensions map.

@katzeforest

This comment has been minimized.

Copy link

@katzeforest katzeforest commented Dec 9, 2019

Thanks for the feedback, I will +1 this to the team on your behalf. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.