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

Add Option to redact subgraph errors in GraphQL response, but not in logs. #3502

Open
Bjohnson131 opened this issue Jul 26, 2023 · 1 comment

Comments

@Bjohnson131
Copy link

Is your feature request related to a problem? Please describe.
It is currently very difficult to find the precise cause of an error when redacting subgraph errors. We know which subgraph the error comes from, but not the error itself. While redacting the subgraph errors in the router's response is necessary for security, we'd still like the error to come as a part of the Router logging.

Describe the solution you'd like
I'd like some sort of configuration option to choose whether subgraph errors are redacted in the response, the logs, or both.

Describe alternatives you've considered
In this case, we already use a log aggregator for all subgraphs, so we can find related logs very quickly, but the error that Apollo Router outputs has new info on the issue that the subgraphs can't have.

@BrynCooke
Copy link
Contributor

Related to #3226, Let's try and bring this in to the overall design of the configuration.

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

No branches or pull requests

3 participants