Etherpad-lite: Etherpad no debería imprimir secuencias de terminales en el registro de archivos

Por favor elabora

El 31 de mayo de 2015 05:36, yurivict [email protected] escribió:

Solo necesita imprimir secuencias de terminales cuando la salida es una terminal real.

-
Responda a este correo electrónico directamente o véalo en Gi

Cuando se envía al archivo, el registro se ve así

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

¿Suena como un error de log4js? Usamos eso ...

El 31 de mayo de 2015 a las 10:46, yurivict [email protected] escribió:

Cuando se envía al archivo, el registro se ve así

Etherpad iniciado ...
^ [[33m [2015-05-30 21: 26: 58.721] Consola [WARN] - ^ [[39mDirtyDB se utiliza. Esto está bien para probar, pero no se recomienda para producción.
^ [[32m [2015-05-30 21: 27: 00.402] [INFO] consola - ^ [[39m Complementos instalados:
^ [[32m [2015-05-30 21: 27: 00.439] [INFO] consola - ^ [[39mInformar errores en https://github.com/ether/etherpad-lite/issues
^ [[32m [2015-05-30 21: 27: 00.442] [INFO] consola - ^ [[39m Su versión de Etherpad es 1.5.6 (b049f4b)
^ [[32m [2015-05-30 21: 27: 00.622] [INFO] consola - ^ [[39m Puede acceder a su instancia de Etherpad en http://0.0.0.0 : 9001 /
^ [[33m [2015-05-30 21: 27: 00.622] [WARN] consola - ^ [[39m El nombre de usuario y la contraseña de administrador no están configurados en settings.json. Para acceder al administrador, descomente y edite 'usuarios' en settings.json

-
Responda a este correo electrónico directamente o véalo en Gi

Este mismo problema surgió con git ~ hace 6 meses, en relación con pager. Es posible que desee buscar esa discusión en archivos. Recuerdo que fue muy difícil adivinar si el buscapersonas puede manejar colores. Terminaron insistiendo en los colores, pero las conjeturas involucradas hacen que el sistema sea propenso a errores. No hay una forma general de hacer esto con precisión.

Tienes que ir por este camino difícil. O simplemente deshabilite los colores cuando la salida no sea el terminal, lo cual es 100% fácil.

Aceptaría una solicitud de extracción si tiene una solución fácil

Ahora, cuando miré, tienes mucho código copiado y pegado con secuencias de terminal.
Necesita usar algún paquete js estándar (como este https://github.com/sindresorhus/chalk) Entonces este paquete manejará la impresión correctamente o será fácil de arreglar. Pero no con el estilo actual codificado de forma rígida.

@yurivict Ver # 2835

Una solución podría ser mirar esto a la inversa: solo renderice en color si se le solicita.