Etherpad-lite: Etherpad não deve imprimir sequências de terminais no arquivo de log

Criado em 31 mai. 2015  ·  8Comentários  ·  Fonte: ether/etherpad-lite

Você só precisa imprimir as sequências de terminal quando a saída for um terminal real.

Minor Bug

Todos 8 comentários

Por favor elabore

Em 31 de maio de 2015, 05:36, yurivict [email protected] escreveu:

Você só precisa imprimir as sequências de terminal quando a saída for um terminal real.

-
Responda a este e-mail diretamente ou visualize-o em Gi tHubhttps: //github.com/ether/etherpad-lite/issues/2685.

Quando canalizado para o arquivo, o log se parece com isto

Started Etherpad...
^[[33m[2015-05-30 21:26:58.721] [WARN] console - ^[[39mDirtyDB is used. This is fine for testing but not recommended for production.
^[[32m[2015-05-30 21:27:00.402] [INFO] console - ^[[39mInstalled plugins:
^[[32m[2015-05-30 21:27:00.439] [INFO] console - ^[[39mReport bugs at https://github.com/ether/etherpad-lite/issues
^[[32m[2015-05-30 21:27:00.442] [INFO] console - ^[[39mYour Etherpad version is 1.5.6 (b049f4b)
^[[32m[2015-05-30 21:27:00.622] [INFO] console - ^[[39mYou can access your Etherpad instance at http://0.0.0.0:9001/
^[[33m[2015-05-30 21:27:00.622] [WARN] console - ^[[39mAdmin username and password not set in settings.json.  To access admin please uncomment and edit 'users' in settings.json

Parece um bug do log4js? Nós usamos isso ...

Em 31 de maio de 2015 10:46, yurivict [email protected] escreveu:

Quando canalizado para o arquivo, o log se parece com isto

Etherpad iniciado ...
^ [[33m [2015-05-30 21: 26: 58.721] [WARN] console - ^ [[39mDirtyDB é usado. Isso é bom para teste, mas não é recomendado para produção.
^ [[32m [2015-05-30 21: 27: 00.402] [INFO] console - ^ [[39m Plugins instalados:
^ [[32m [2015-05-30 21: 27: 00.439] [INFO] console - ^ [[39mReport bugs em https://github.com/ether/etherpad-lite/issues
^ [[32m [2015-05-30 21: 27: 00.442] [INFO] console - ^ [[39m Sua versão Etherpad é 1.5.6 (b049f4b)
^ [[32m [2015-05-30 21: 27: 00.622] [INFO] console - ^ [[39mVocê pode acessar sua instância Etherpad em http://0.0.0.0 : 9001 /
^ [[33m [2015-05-30 21: 27: 00.622] [WARN] console - ^ [[39m nome de usuário e senha de administrador não definidos em settings.json Para acessar o administrador, descomente e edite 'usuários' em settings.json

-
Responda a este e-mail diretamente ou visualize-o em Gi tHubhttps: //github.com/ether/etherpad-lite/issues/2685#issuecomment -107150769.

Este mesmo problema surgiu com o git cerca de 6 meses atrás, em relação ao pager. Você pode querer pesquisar essa discussão nos arquivos. Lembro que era muito difícil adivinhar se o pager pode lidar com cores. Eles acabaram insistindo nas cores, mas o trabalho de adivinhação envolvido torna o sistema sujeito a erros. Nenhuma maneira geral de fazer isso com precisão.

Você precisa seguir esse caminho difícil. Ou apenas desative as cores quando a saída não for o terminal, o que é 100% fácil.

Eu aceitaria uma solicitação pull se você tiver uma solução fácil

Agora, quando olhei, você tem muito código copiado e colado com sequências terminais.
Você precisa usar algum pacote js padrão (como este https://github.com/sindresorhus/chalk) Então este pacote irá lidar com a impressão corretamente ou será fácil de consertar. Mas não com o estilo codificado atual.

@yurivict Veja # 2835

Uma solução pode ser olhar para isso ao contrário: apenas renderize em cores se for solicitado.

Esta página foi útil?
0 / 5 - 0 avaliações