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.

Sign up for GitHub

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

Jump to bottom

Fix error handling when parsing variables parameter #130

Closed
nnance opened this issue Sep 9, 2016 · 0 comments · Fixed by #131
Closed

Fix error handling when parsing variables parameter #130

nnance opened this issue Sep 9, 2016 · 0 comments · Fixed by #131
Assignees
@nnance
Milestone
0.3

Comments

@nnance
Copy link
Contributor

nnance commented Sep 9, 2016

When a string is passed as the variable parameter in a query the integrations are not handling the possibility of a parse error. This is currently marked as a TODO in each integration
@nnance nnance added this to the 0.3 milestone Sep 9, 2016
@nnance nnance self-assigned this Sep 9, 2016
@nnance nnance mentioned this issue Sep 9, 2016
Merged
4 tasks
@nnance nnance changed the title Add error handling for variables string Fix error handling when parsing variables parameter Sep 9, 2016
trevor-scheer pushed a commit that referenced this issue May 6, 2020
@renovate
chore(deps): update dependency tslint to v5.17.0 (#130)
dcad89b
trevor-scheer pushed a commit that referenced this issue May 14, 2020
@renovate
chore(deps): update dependency tslint to v5.17.0 (#130)
0663389
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nnance nnance

Labels
None yet
Projects
None yet
Milestone
0.3
Development

Successfully merging a pull request may close this issue.

Handle variable parse error apollographql/apollo-server
2 participants
@zol @nnance