kevva / kevva/graphql-got

Swallows GraphQL errors

Open
#3 4 comments 0 reactions 0 assignees View on GitHub
bug help wanted
Dominant language
JavaScript
Stars
103
Forks
5
PR merge metrics
No merged PRs in 30d

Description

If your GraphQL endpoint returns an error, this module will swallow it here: https://github.com/kevva/graphql-got/blob/v0.1.1/index.js#L38

Instead of dropping `res.body.errors`, they should be assigned to `res` (maybe as `res.errors`?). Would you accept a PR that introduces this change?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at index.js line 38, where the GraphQL response currently drops res.body.errors, and trace the surrounding response handling. The work is done when those errors remain available on the returned response and the repository’s existing tests confirm the behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
graphql, javascript
Domain
api
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.