mirror of
https://github.com/hoppscotch/hoppscotch.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #4395] [bug]: Missing status in GraphQL requests #1606
Labels
No labels
CodeDay
a11y
browser limited
bug
bug fix
cli
core
critical
design
desktop
discussion
docker
documentation
duplicate
enterprise
feature
feature
fosshack
future
good first issue
hacktoberfest
help wanted
i18n
invalid
major
minor
need information
need testing
not applicable to hoppscotch
not reproducible
pull-request
question
refactor
resolved
sandbox
self-host
spam
stale
testmu
wip
wont fix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hoppscotch#1606
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @rb090 on GitHub (Oct 1, 2024).
Original GitHub issue: https://github.com/hoppscotch/hoppscotch/issues/4395
Originally assigned to: @barrettluke on GitHub.
Is there an existing issue for this?
Current behavior
When firing a GraphQL request the Http Status is missing in the UI:
When firing a REST request the request status is displayed beautifully:
Can this behaviour for GraphQL request be turned on and off somewhere?
Steps to reproduce
Environment
Production
Version
Local
@barrettluke commented on GitHub (Oct 2, 2024):
I would like to work on this.
@nivedin commented on GitHub (Oct 3, 2024):
@barrettluke assigning this to you
@rb090 commented on GitHub (Oct 3, 2024):
Thank you so much, I really appreciate your help with this. If there is sth I can help with, please let me know. I am using atm the AIO container of the hoppscotch community edition. So can that one please get a new release with this fix once it is done?
We host hoppscotch on our side with that Docker container.
@Narasimha677 commented on GitHub (Oct 6, 2024):
hey @barrettluke , can i know are u still working on this issue?
@barrettluke commented on GitHub (Oct 7, 2024):
@Narasimha677 yes I am still working on this.
@barrettluke commented on GitHub (Oct 10, 2024):
Should this show up when Connect button is pressed or a connections is made or should this show up when Request button is pressed and query is sent?
@rb090 commented on GitHub (Oct 10, 2024):
I would like to see it when the GraphQL request was send and we got a response from the GQL server. So basically like it is for REST requests 🙂. Like this when sending GQL requests in P*****n 🙈.
@barrettluke commented on GitHub (Oct 11, 2024):
After reviewing the code, I noticed that RequestRunner.ts—specifically the runRESTRequest$ function—is specific for REST requests. Given this, I don’t think it should be used for GraphQL queries as-is.
If a new file or function is needed to handle the checks for GraphQL requests, a key question is: how should the response object look? Should the new data be integrated into the current GraphQL response, or would it make more sense to nest it within the existing structure? Additionally, how much data would you like returned? Should we limit it to the status code, response time (ms), and size, or should we include all the data returned by an HTTP response?
It seems like there are a few important questions to address, and this could potentially become part of a larger task. An alternative could be to quickly implement a basic solution with a curl request, but I don’t think that would be ideal for maintainability.
@rb090 commented on GitHub (Oct 11, 2024):
Oh no, I am sorry that this causes so much complexity. For my team and me this would be really a super helpful functionality 😢.
I guess that is a question on the hoppscotch code?
The HTTP status code is an attribute outside the GQL response from the server point of view:
That would be super helpful tbh. And make us more than happy.
If server returns a response on a failed GQL request, this response is already visible in the UI and this is super cool.
Other data like fe. response headers would be not so important for us.
@ToshY commented on GitHub (Jan 1, 2025):
I'm experiencing the same problem. Is there any hold-up with the existing PR #4435 ?
@liyasthomas commented on GitHub (Jul 30, 2025):
This issue has been resolved in the latest version.