LogViewer doesn't handle ANSI "CSI Ps K" (Erase in Line) control character
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Start at the LogViewer entry point and trace how ANSI control characters are parsed, focusing on CSI Ps K (Erase in Line). Use the JavaScript reproduction in the issue as the first check; done means the secret message is rendered without the duplicated shell prompt or unintended hidden text.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When you debug a node ( Compute -> Nodes -> SomeNode -> Terminal, or via CLI oc debug node/[nodeName] ) and you open a separate console window where you open up the spawned debug pods logs, the shell prompt (e.g sh-5.1#) is shown twice at the start of the logs.
This is not shown when oc logs is used because ANSI control characters are used to hide the first sh-5.1.
reproducing example:
import { LogViewer } from '@patternfly/react-log-viewer';
// shows "secret message"
export default () => (
[LogViewer data="\x1b[0msecret message\r\x1b[K" /]
);
vs.
h1. shows nothing
echo -e "\x1b[0msecret message\r\x1b[K"
xref https://redhat.atlassian.net/browse/OCPBUGS-54247
Jira Issue: PF-3123
- Lenguaje dominante
- TypeScript
- Estrellas
- 34
- Forks
- 19
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de patternfly/react-log-viewer
-
Search within Log viewer can cause the parent container to shift and break the layout, when embedded in a scrollable parentPosiblemente ocupada @fao89 la tomó hace 139 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 67/100
patternfly/react-log-viewer#106 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
patternfly/react-log-viewer#98 · 1 comentario ·
-
How to set search text by props?Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
patternfly/react-log-viewer#95 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
patternfly/react-log-viewer#91 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
patternfly/react-log-viewer#85 · 1 comentario ·
Todos los issues de patternfly/react-log-viewer
Issues similares
-
fix(frontend): correct VITE_IS_DEVELPOMENT typo in shared/app.ts and align dev environment flagsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
vicharanashala/ajrasakha#1439 ·
Los mantenedores suelen responder en 1 día
-
Add: TV5Monde AsieAbiertoapproved check:passed streams:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
codex-team/editor.js#3037 ·
-
complexity: trivial good first issue impact: low
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
ueberdosis/tiptap#8522 ·
Los mantenedores suelen responder en 1 día
-
Bump anybuf version to ^1.0.0Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
wormhole-foundation/wormhole#5062 ·
Los mantenedores suelen responder en 1 día