balderdashy/sails

Preserve async hook on call to helper

Open

#6.960 aberto em 12 de mar. de 2020

Ver no GitHub
 (4 comments) (1 reaction) (0 assignees)JavaScript (1.953 forks)batch import
help wantedmore info please

Métricas do repositório

Stars
 (22.778 stars)
Métricas de merge de PR
 (Mesclagem média 23h 44m) (1 fundiu PR em 30d)

Description

So there is this thing called async hooks in Node now that allow for stuff to be preserved like you would with thread local in other languages. A popular library that uses that is cls-hooked. This is very useful to put trace ID and add that trace to the logs when you want to do distributed logging (or add a user ID to have some context for your logs). However, the machine that powers sails messes up with this process and we lose the information when an helper is called. I am not sure what happens, but it is most likely related to how the machine that powers sails works (by making HTTP calls I believe for each helper). I think @mikermcneil would be the one to answer that since not a lot people seem to have touched this code except him...

Guia do colaborador

Direção de pesquisa
Investigue como o módulo machine subjacente no Sails lida com hooks assíncronos. O problema sugere que a machine faz chamadas HTTP para cada helper, o que pode quebrar o contexto assíncrono. Examine o código da machine e qualquer código de invocação de helper relacionado para entender onde o contexto é perdido. Verifique os 4 comentários na issue para obter pistas adicionais. Considere entrar em contato com @mikermcneil para esclarecimentos sobre o comportamento pretendido de propagação de hooks assíncronos. Explore se uma correção envolveria envolver chamadas de helper para preservar o contexto do hook assíncrono usando bibliotecas como cls hooked ou similares.
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-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
Node.js async hooksSails MVC framework
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
20