cube-js/cube

Continue wait for GraphQL responses with http status code 500.

Aperta

#6651 aperta il 24 mag 2023

Vedi su GitHubGuida contributor
 (4 commenti) (2 reazioni) (1 assegnatario)Rust (1965 fork)batch import
api:graphqlhelp wanted

Metriche repository

Star
 (19.563 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Analizza il gestore delle risposte GraphQL in Cube.js e assicurati che le risposte di attesa continua restituiscano HTTP 200 invece di 500 per gli endpoint GraphQL, simile al comportamento dell'API REST.
Tech stack
Nessuno
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GraphQLHTTP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70