coolaj86/greenlock.js-ARCHIVED
3
3
Derivar 0
Código Questões 6 Pedidos de integração 1 Lançamentos Trabalho

Function skipIfNeedBe in http-middleware.js should return a value #46

Questão nova
fechada(s)
2019-11-06 09:19:33 +00:00 aberta(s) por Ghost · 3 comentários
Ghost comentou 2019-11-06 09:19:33 +00:00
Copiar ligação

Hi AJ,

first of all, thank you for your awesome work!
Testing around with the new version of greenlock I stumbled across the following exception:

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
node_modules/@root/greenlock-express/http-middleware.js:62:9

If I understood your code correctly, I'd suggest to add a return value to the function skipIfNeedBe of http-middleware.js if the conditions are met.

Hi AJ, first of all, thank you for your awesome work! Testing around with the new version of greenlock I stumbled across the following exception: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client node_modules/@root/greenlock-express/http-middleware.js:62:9 If I understood your code correctly, I'd suggest to add a return value to the function skipIfNeedBe of http-middleware.js if the conditions are met.
Ghost comentou 2019-11-06 09:21:41 +00:00
Autor(a)
Copiar ligação

I just realized, that I used the wrong repository to open the issue. Sorry for that!

I just realized, that I used the wrong repository to open the issue. Sorry for that!
coolaj86 comentou 2019-11-06 15:49:43 +00:00
Proprietário(a)
Copiar ligação

If you rm -rf node_modules and reinstall, I believe the problem was fixed yesterday:

https://git.coolaj86.com/coolaj86/greenlock-express.js/src/branch/master/http-middleware.js

If you `rm -rf node_modules` and reinstall, I believe the problem was fixed yesterday: https://git.coolaj86.com/coolaj86/greenlock-express.js/src/branch/master/http-middleware.js
Ghost comentou 2019-11-06 17:34:34 +00:00
Autor(a)
Copiar ligação

Yes, you are right. The new version does not have this issue. Thanks alot!

Yes, you are right. The new version does not have this issue. Thanks alot!
Ghost encerrou esta questão 2019-11-06 17:34:34 +00:00
Inicie a sessão para participar neste diálogo.
Sem ramo ou etiqueta especificados
Ramos Etiquetas
Rótulos
Sem itens
Sem rótulo
Etapa
Sem itens
Sem etapa
Encarregados
Retirar todos os encarregados
jshaver coolaj86
Sem encarregados
2 Participantes
Notificações
Data de vencimento
Sem data de vencimento definida.
Dependências

Não estão definidas dependências.

Referência: coolaj86/greenlock.js-ARCHIVED#46
Nenhuma descrição fornecida.
Eliminar o ramo "%!s()"

Eliminar um ramo é algo permanente. Embora o ramo eliminado possa continuar a existir por um breve período de tempo antes de ser realmente removido, a operação NÃO PODERÁ ser desfeita na maioria dos casos. Quer continuar?