cube-js/cube

Continue wait for GraphQL responses with http status code 500.

Open

#6.651 geöffnet am 24. Mai 2023

Auf GitHub ansehen
 (4 Kommentare) (2 Reaktionen) (1 zugewiesene Person)Rust (1.965 Forks)batch import
api:graphqlhelp wanted

Repository-Metriken

Stars
 (19.563 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 16h) (138 gemergte PRs in 30 T)

Beschreibung

Describe the bug Continue wait error for GraphQL responses with http status code 500.

To Reproduce Steps to reproduce the behavior:

  1. Make a graphql query which would take more than 5 seconds (or reduce continueWait time configuration).

Expected behavior Though, we don't have this documented for graphql. for rest api it is documented to return with 200. Can we please have same http status code for graphql also. https://cube.dev/docs/http-api/rest/#prerequisites-continue-wait

Additional context From Slack thread- https://cube-js.slack.com/archives/C04NYBJP7RQ/p1684846190185819

@vasilev-alex

Contributor Guide

Research-Richtung
Untersuchen Sie den GraphQL Antworthandler in Cube.js und stellen Sie sicher, dass Continue Wait Antworten für GraphQL Endpunkte HTTP 200 anstelle von 500 zurückgeben, ähnlich wie beim REST API Verhalten.
Tech Stack
Keine
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GraphQLHTTP
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70