drogonframework/drogon

Header "connection: close" disconnects the client before Response Stream is started

Open

#2276 aperta il 10 mar 2025

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (1014 fork)batch import
help wanted

Metriche repository

Star
 (10.462 star)
Metriche merge PR
 (Merge medio 2g 2h) (7 PR mergiate in 30 g)

Descrizione

Describe the bug

In our project we use Drogon for HTTP response streaming. However, when client includes connection header that is not keep-alive, response stream is immediaitely closed.

Source code: https://github.com/drogonframework/drogon/blob/cbf63f8fc4d849bbb82eeb1c83fcf8ff953f19f3/lib/src/HttpServer.cc#L981-L990

It is common practice to use connection: close to indicate that connection should be closed after processing is finished*, meaning - after HTTP response stream is done.

In our setup, we needed to patch drogon like this: https://github.com/openvinotoolkit/model_server/pull/3113/commits/8ca2918c04ae9fab77ad157dcd23673971d84420

To Reproduce Steps to reproduce the behavior:

  1. Start drogon with streaming response endpoint defined
  2. Request with any client including header connection: close
  3. Request is immediately rejected

Expected behavior Connection is closed after response streaming is finished.

Desktop (please complete the following information):

  • OS: ubuntu, redhat, windows
  • Client: any

More context We are using Continue VSCode plugin as a client for code completions: https://github.com/continuedev/continue We use drogon to serve http streaming (code completions), however, this plugin (client) sends connection: close header.

Guida contributor

Direzione di ricerca
Il bug si trova in HttpServer.cc alle righe 981-990. Il problema descrive che quando il client invia 'connection: close', il flusso di risposta viene chiuso immediatamente invece che dopo il completamento dello streaming. Nella pull request collegata (https://github.com/openvinotoolkit/model server/pull/3113/commits/8ca2918c04ae9fab77ad157dcd23673971d84420) è fornita una patch che mostra le modifiche esatte necessarie. Rivedi questa patch e applica una correzione simile a HttpServer.cc di Drogon, assicurandoti che la connessione venga chiusa solo dopo che il flusso di risposta è completo.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++ knowledgeHTTP protocol basicsDrogon framework basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50