Winston: No se puede instalar en la ventana acoplable después de 3.3.0/3.3.1

El mismo problema aquí, pero no solo en la ventana acoplable. No puedo npm instalar mi proyecto debido a esta dependencia:

npm ERR! Error while executing:
npm ERR! /usr/local/bin/git ls-remote -h -t ssh://[email protected]/DABH/diagnostics.git
npm ERR! 
npm ERR! ssh: connect to host github.com port 22: Connection timed out
npm ERR! fatal: Could not read from remote repository.
npm ERR! 
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR! 
npm ERR! exited with error code: 128

Me pregunto si el problema es que está tratando de llegar a la profundidad a través de ssh en lugar de https.

¿Puede uno de ustedes intentar modificar el package.json para que el diagnostics dep sea
git+https://github.com/DABH/diagnostics.git
? Eso debería forzarlo a usar https, pero quiero estar 100 % seguro de que eso resuelve tu problema antes de hacer otro lanzamiento de hotifx. ¡Gracias!

Tengo el mismo problema al instalar a través de Docker.
Intenté modificar el paquete.json, pero no estoy seguro de si lo estoy haciendo bien.
en node_modules/winston/package.json cambié: "diagnostics": " github:DABH/diagnostics#master ",
en: "diagnósticos": "git+ https://github.com/DABH/diagnostics.git ",

Esto no funciona, pero esperaba necesitar cambiar una dependencia que termina con "diagnostics.git", pero no pude encontrar eso en el proyecto.

editar: estoy usando ' node:12.10.0-alpine ' como imagen base para docker

He estado investigando sobre el problema y parece que el problema proviene de la imagen acoplable basada en alpine. Incluso el siguiente Dockerfile falla al compilarse, independientemente de la versión del nodo.

FROM node:14-alpine

RUN npm install git+https://github.com/lodash/lodash

Ah, debido a que la dependencia proviene de github, necesitas git...

FROM node:14-alpine

RUN apk update && apk upgrade && \
    apk add --no-cache bash git openssh

RUN npm install git+https://github.com/lodash/lodash

Entiendo que no es una gran solución. La alternativa es que tengo que publicar mi bifurcación diagnostics en NPM, pero ese paquete ya está en NPM, así que supongo que tendría que cambiar el nombre de mi repositorio o algo así para poder publicar de forma única. Cualquier consejo es bienvenido, de lo contrario lo veré más tarde hoy.

Sí, se me ocurrió la misma solución (ni siquiera podía imaginar que git no se envía en alpine ...) Estoy bien con esa solución.

El problema no era Winston, así que doy por cerrado esto. ¡Gracias por la ayuda!

hola, lo mismo por mi parte, algunos registros:

https://travis-ci.org/github/weareopensource/Node/builds/700773943
https://hub.docker.com/repository/registry-1.docker.io/weareopensource/node/builds/cd59b608-6976-48c2-8765-e7076afb971f

Gracias, mira mi publicación arriba ^^

@DABH @Kivol

hum, sí, entiendo, gracias :)

Para mí la solución sigue siendo temporal y el tema no debe cerrarse.

Un paquete NPM no debería requerir git para la instalación, sino solo npm. Si usa un TENEDOR de otro paquete porque este no corresponde a su necesidad. Creo que este FORK debe integrarse en la fuente o convertirse en un proyecto mantenido por Winston y, por lo tanto, publicado como sugirió. Sería bastante extraño imponer la instalación de git o cualquier otra herramienta para la instalación de un paquete npm, por cientos de imágenes de docker. Las imágenes y el proceso deben ser lo más ligeros posible.

Hola, solo quiero expresar que git no debería ser un dep para winstonjs. Espero que esto se pueda resolver. Tenemos algunos cientos de imágenes acoplables y no creo que modificarlas todas sea muy ideal. Y estoy seguro de que muchos de nosotros sentiríamos lo mismo. :)

Si 3.3.x realmente tiene un cambio tan importante, deberíamos moverlo a 4.x en su lugar.

@DABH @Kivol

En segundo lugar, los comentarios anteriores acerca de que este cambio es un cambio radical. Tenemos imágenes acoplables alpinas en producción que se romperán debido a esto.

Si esto es necesario, sería compatible con un cambio 4.x.

Wow, este rompe el internet... :)
Muy extraño requerir git de hecho.

Tengo el mismo problema no en una imagen acoplable sino en mi red empresarial.

Uso artefactos para descargar todas las dependencias (npm y github fueron bloqueados). Si usa su bifurcación, todos los proyectos sin acceso público fallarán. El mayor impacto es para empresas o CI/CD con acceso restringido.

Desde mi punto de vista, si desea mantener su bifurcación, debe ser estándar y crear un nuevo paquete npm y no referirse a un repositorio de github.

Entonces, ¿por qué este boleto todavía está cerrado?

Mismo problema aquí, por favor solucionarlo. Desafortunadamente, winston está instalado como un módulo de una subdependencia, por lo que no podemos modificar directamente la versión. Esto rompe nuestra canalización de CI que no tiene acceso a github público.

+1 para este problema, esto rompe nuestra canalización de CI.
Por favor, deshaga los cambios.

@Kivol , vuelva a abrir, instalar git no es una opción para muchos proyectos
especialmente cuando winston no es dependencia directa.

+1 también. Esto viola el principio de construir una vez y rompe las construcciones para nuestros consumidores. ¿Tal vez revertir hasta que se solucione sea una buena opción aquí?

Solo para decir que creo que los comentarios anteriores muestran claramente cuánto se usa Winston y depende de muchos proyectos. Entonces, si bien es doloroso cuando ocurren problemas, espero hablar por muchos, cuando digo que valoramos la contribución que hacen Winston y sus desarrolladores.

Aquí igual.
Dos puntos para agregar:

1. agregar dependencias al trabajo de CI/CD basado en una imagen de Alpine rompe el concepto y tiene un costo (piense en "_green dev_")
2. en un entorno corporativo, no podemos cambiar la configuración general de la red (¡si alguna vez pudiéramos!) para obtener un solo paquete

Gracias por los comentarios activos, tenga paciencia durante un par de horas mientras tratamos de implementar una solución mejor (no dependiente de git) para esto.

Todo, se ha lanzado v3.3.2 , esto descarga la dependencia bifurcada diagnostics de NPM, no de GitHub, por lo que debería funcionar para usted. Por favor, siéntase libre de confirmar o negar. ¡Gracias!

Gracias por la solución.