Skip to content
This repository was archived by the owner on Dec 19, 2019. It is now read-only.

Implement exception logging #220

Closed
paliarush opened this issue Oct 24, 2018 · 0 comments
Closed

Implement exception logging #220

paliarush opened this issue Oct 24, 2018 · 0 comments
Labels
framework Quality improvements
Milestone
Release: 2.3.4

Comments

@paliarush
Copy link
Contributor

paliarush commented Oct 24, 2018 •
edited
Loading

In case when exception is thrown from the resolver, in addition to being displayed to the client, the exception must be logged.

Server-side exceptions must be logged as errors; while client-side exceptions should have less strict level.
@novikor novikor self-assigned this Jan 31, 2019
novikor pushed a commit to novikor/graphql-ce that referenced this issue Feb 7, 2019
Implement exception logging magento#220.
583ff11 
Created custom error handler;
Applied default error handling functionality
novikor pushed a commit to novikor/graphql-ce that referenced this issue Feb 7, 2019
Implement exception logging magento#220.
9027c50 
Implemented client and server errors logging
@novikor novikor mentioned this issue Feb 18, 2019
Merged
4 tasks
@naydav naydav added this to the Release: 2.3.3 milestone May 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
framework Quality improvements
Projects
None yet
Milestone
Release: 2.3.4
Development

No branches or pull requests
4 participants
@naydav @paliarush @lenaorobei @novikor