Etherpad-lite: Etherpad ne doit pas imprimer les séquences de terminal dans le journal des fichiers

Veuillez préciser

Le 31 mai 2015 à 05h36, yurivict [email protected] a écrit:

Vous ne devez imprimer les séquences de terminaux que lorsque la sortie est un terminal réel.

-
Répondez directement à cet e-mail ou consultez-le sur Gi tHubhttps: //github.com/ether/etherpad-lite/issues/2685.

Lorsqu'il est redirigé vers un fichier, le journal ressemble à ceci

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

Cela ressemble à un bogue de log4js? Nous utilisons ça ...

Le 31 mai 2015 à 10h46, yurivict [email protected] a écrit:

Lorsqu'il est redirigé vers un fichier, le journal ressemble à ceci

Démarrage d'Etherpad ...
^ [[33m [2015-05-30 21: 26: 58.721] [WARN] console - ^ [[39mDirtyDB est utilisé. C'est bien pour les tests mais pas recommandé pour la production.
^ [[32m [2015-05-30 21: 27: 00.402] [INFO] console - ^ [[39mInstalled plugins:
^ [[32m [2015-05-30 21: 27: 00.439] [INFO] console - ^ [[39mReportez les bogues sur https://github.com/ether/etherpad-lite/issues
^ [[32m [2015-05-30 21: 27: 00.442] [INFO] console - ^ [[39mVotre version Etherpad est 1.5.6 (b049f4b)
^ [[32m [2015-05-30 21: 27: 00.622] [INFO] console - ^ [[39mVous pouvez accéder à votre instance Etherpad à http://0.0.0.0 : 9001 /
^ [[33m [2015-05-30 21: 27: 00.622] [WARN] console - ^ [[39mAdmin nom d'utilisateur et mot de passe non définis dans settings.json. Pour accéder à l'administrateur, décommentez et modifiez les `` utilisateurs '' dans settings.json

-
Répondez directement à cet e-mail ou consultez-le sur Gi tHubhttps: //github.com/ether/etherpad-lite/issues/2685#issuecomment -107150769.

Ce même problème est survenu avec git il y a environ 6 mois, en relation avec le pager. Vous voudrez peut-être consulter cette discussion dans les archives. Je me souviens que c'était très difficile de deviner si le téléavertisseur pouvait gérer les couleurs. Ils ont fini par insister sur les couleurs, mais le travail de conjecture impliquée rend le système sujet aux erreurs. Aucun moyen général de le faire avec précision.

Vous devez soit aller aussi dur. Ou désactivez simplement les couleurs lorsque la sortie n'est pas le terminal, ce qui est 100% facile.

J'accepterais une demande de tirage si vous avez une solution simple

Maintenant, quand j'ai regardé, vous avez beaucoup de code copié-collé avec des séquences terminales.
Vous devez utiliser un package js standard (comme celui-ci https://github.com/sindresorhus/chalk). Ensuite, soit ce package gérera correctement l'impression, soit il sera facile de le réparer. Mais pas avec le style codé en dur actuel.

@yurivict Voir # 2835

Une solution pourrait être de regarder cela à l'envers: ne rendre en couleur que si cela vous est demandé.