Client.exec returns code null when command code is 0
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- node.js, typescript
- Área
- backend, networking
Línea de trabajo
Comienza en src/index.ts alrededor de la línea 361 e inspecciona cómo client.exec gestiona el evento channel.on('exit'). Reproduce el problema con la prueba fallida 'exec should return correct code' que se muestra en el informe; se considera terminado cuando un comando echo exitoso devuelve result.code === 0 y conserva la stdout esperada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When a command returns 0, node-ssh's exec always returns code: null.
It seems the faulty line is this one.
Here's a PR to fix that.
Even though this line is incorrect (the nullish coalescing operator should be used instead for example), it appears that it's not the only problem: channel.on('exit' is never emitted.
A simple way to reproduce it is with this (failing) test:
sshit('exec should return correct code', async function (t, port, client) {
await connectWithPassword(port, client)
const result = await client.exec('echo', ['$some', 'S\\Thing', '"Yo"'], { stream: 'both' })
t.is(result.stdout, '$some S\\Thing "Yo"')
t.is(result.code, 0)
})
Expected
result.code === 0
Actual
result.code === null
- Lenguaje dominante
- TypeScript
- Estrellas
- 1k
- Forks
- 94
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 steelbrain/node-ssh
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
steelbrain/node-ssh#487 ·
-
ssh-certificate supportAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
steelbrain/node-ssh#482 · 3 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
steelbrain/node-ssh#474 · 3 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
steelbrain/node-ssh#473 · 1 comentario · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
steelbrain/node-ssh#472 ·
Todos los issues de steelbrain/node-ssh
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
cli: `execCargoMetadata` decodes stdout per chunk, corrupting non-ASCII paths across pipe boundariesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
lingdojo/kana-dojo#34127 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
osmlab/name-suggestion-index#13104 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100