restify/node-restify

Gzip plugin prevents text/event-stream from working

Open

#820 aberto em 2 de jun. de 2015

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)JavaScript (10.695 stars) (983 forks)batch import
BugCriticalHelp WantedUnconfirmed

Description

I have to call

      res.handledGzip && res.handledGzip();
      res.removeHeader('Content-Encoding');

in my SSE plugin to bypass the Gzip plugin—otherwise EventSource fails.

The solution would involve adding support for keep alive connection in the Gzip plugin.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
backend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Node.js basicsUnderstanding of HTTP streamingRestify plugin system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40
Direção de pesquisa
Investigate the restify Gzip plugin implementation (likely in lib/plugins/gzip.js). The issue suggests that adding support for keep alive connections in the Gzip plugin would fix SSE. Look for current handling of 'Content Encoding' header and streaming responses. Consider how other frameworks handle gzip with SSE, and test with EventSource.