Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Log errors #167

Open
slonoed opened this issue Mar 27, 2019 · 2 comments
Open

Log errors #167

slonoed opened this issue Mar 27, 2019 · 2 comments

Comments

@slonoed
Copy link

slonoed commented Mar 27, 2019

I want to continue a discussion from this PR #166

The previous version has the ability to specify formatError function, which allows to log errors and used by me only for logging.
The new version doesn't have anything to log errors.
This is the critical feature.

I want to get an official decision will it be implemented (in some way or another). If yes, when?

As a workaround, I can implement my custom wrapper on top of all resolvers.

@ganemone
Copy link
Contributor

Thank you for reaching out! The solution we are going to implement is logging via graphql-middleware. Originally, I was planning on building that into this plugin, however it actually works better as an enhancer on the GraphQLSchemaToken. I'm working on building one right now, so we will have a solution very soon. I will put you as a subscriber on the diff.

@slonoed
Copy link
Author

slonoed commented Mar 29, 2019

@ganemone please, ping me if you need help with this. Implementation or testing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants