Winston: Tidak dapat menginstal di buruh pelabuhan setelah 3.3.0/3.3.1

Masalah yang sama di sini, tetapi tidak hanya di buruh pelabuhan. Saya tidak dapat menginstal npm proyek saya karena ketergantungan ini:

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

Bertanya-tanya apakah masalahnya adalah mencoba menjangkau dep melalui ssh alih-alih https.

Bisakah salah satu dari Anda mencoba memodifikasi package.json winston sehingga diagnostics dep menjadi
git+https://github.com/DABH/diagnostics.git
? Itu harus memaksanya untuk menggunakan https tetapi saya ingin 100% yakin bahwa itu menyelesaikan masalah Anda sebelum melakukan rilis hotifx lainnya. Terima kasih!

Saya mengalami masalah yang sama ketika menginstal melalui buruh pelabuhan.
Saya mencoba memodifikasi package.json, tetapi saya tidak yakin apakah saya melakukannya dengan benar.
di node_modules/winston/package.json saya mengubah: "diagnostics": " github:DABH/diagnostics#master ",
ke: "diagnostik": "git+ https://github.com/DABH/diagnostics.git ",

Ini tidak berfungsi, tetapi saya berharap perlu mengubah ketergantungan yang diakhiri dengan "diagnostics.git" tetapi saya tidak dapat menemukannya di proyek.

edit: Saya menggunakan ' node:12.10.0-alpine ' sebagai gambar dasar untuk buruh pelabuhan

Saya telah menyelidiki tentang masalah ini dan sepertinya masalahnya berasal dari gambar buruh pelabuhan berbasis alpine. Bahkan Dockerfile berikut gagal dibuat, terlepas dari versi simpulnya.

FROM node:14-alpine

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

Ah, karena dep berasal dari github, Anda perlu 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

Saya mengerti itu bukan solusi yang bagus. Alternatifnya adalah saya harus mempublikasikan garpu diagnostics saya di NPM, tetapi paket itu sudah ada di NPM jadi saya kira saya perlu mengubah nama repo saya atau sesuatu agar saya dapat menerbitkan secara unik. Setiap tips, selamat datang, saya akan melihat ini nanti hari ini.

Ya, saya telah menemukan solusi yang sama. (Saya bahkan tidak dapat membayangkan bahwa git tidak dikirimkan di alpine..) Saya setuju dengan solusi itu.

Masalahnya bukan dengan winston jadi saya menutup ini. Terima kasih untuk bantuannya!

halo, sama di sisi saya, beberapa log:

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

Terima kasih, lihat posting saya di atas ^^

@DABH @Kivol

hmm, ya, saya mengerti, thx :)

Bagi saya solusinya tetap sementara dan masalahnya tidak boleh ditutup.

Paket NPM seharusnya tidak memerlukan git untuk instalasi tetapi hanya npm. Jika Anda menggunakan FORK dari paket lain karena yang ini tidak sesuai dengan kebutuhan Anda. Saya pikir FORK ini harus diintegrasikan pada sumbernya atau menjadi proyek yang dikelola untuk Winston dan oleh karena itu diterbitkan seperti yang Anda sarankan. Akan sangat aneh untuk memaksakan instalasi git atau alat lain untuk instalasi paket npm, untuk ratusan gambar buruh pelabuhan. Gambar dan proses harus seringan mungkin

Hai, saya hanya ingin menyatakan bahwa git tidak boleh menjadi dep untuk winstonjs. Semoga ini bisa diselesaikan. Kami memiliki beberapa ratus gambar buruh pelabuhan dan saya tidak berpikir memodifikasi semuanya akan sangat ideal. Dan saya yakin banyak dari kita akan merasakan hal yang sama. :)

Jika 3.3.x benar-benar memiliki perubahan yang melanggar seperti itu, kita harus memindahkannya ke 4.x sebagai gantinya.

@DABH @Kivol

Kedua, komentar di atas tentang perubahan ini sebagai perubahan yang melanggar. Kami memiliki gambar buruh pelabuhan alpine dalam produksi yang akan rusak karena ini.

Jika ini diperlukan, maka akan mendukung perubahan 4.x.

Wow, yang ini merusak internet ... :)
Sangat aneh untuk membutuhkan git memang.

Saya memiliki masalah yang sama bukan pada gambar buruh pelabuhan tetapi di jaringan perusahaan saya.

Saya menggunakan artifactory untuk mengunduh semua dependensi (npm dan github diblokir). Jika Anda menggunakan garpu, semua proyek tanpa akses publik akan gagal. Dampak terbesar bagi perusahaan atau CI/CD dengan restreint access.

Untuk sudut pandang saya jika Anda ingin mempertahankan garpu Anda, Anda harus menjadi standar dan membuat paket baru npm dan tidak merujuk ke repo github.

Jadi mengapa tiket ini masih ditutup?

Masalah yang sama di sini, tolong perbaiki. Sayangnya winston diinstal sebagai modul dari subdependensi, jadi kami tidak dapat langsung mengubah versinya. Ini merusak saluran CI kami yang tidak memiliki akses ke github publik.

+1 untuk masalah ini, ini merusak saluran CI kami.
Harap kembalikan perubahan.

@Kivol tolong buka kembali, instal git bukan pilihan untuk banyak-banyak proyek
terutama ketika winston bukan ketergantungan langsung.

+1 juga. Ini melanggar prinsip build sekali dan merusak build untuk konsumen kami. Mungkin kembali sampai diperbaiki adalah pilihan yang baik di sini?

Hanya untuk mengatakan bahwa saya pikir komentar di atas menunjukkan dengan jelas berapa banyak Winston digunakan dan tergantung pada banyak proyek. Jadi, meskipun menyakitkan ketika masalah terjadi - saya harap saya berbicara mewakili banyak orang - ketika saya mengatakan bahwa kami menghargai kontribusi Winston dan para pengembangnya.

Sama disini.
Dua poin untuk ditambahkan:

1. menambahkan dependensi ke pekerjaan CI/CD berdasarkan gambar Alpine merusak konsep dan memiliki biaya (pikirkan "_green dev_")
2. di lingkungan perusahaan, kami tidak dapat mengubah konfigurasi jaringan secara keseluruhan (jika kami bisa!) untuk mendapatkan hanya satu paket

Terima kasih atas komentar aktifnya, harap bersabar selama beberapa jam saat kami mencoba meluncurkan solusi yang lebih baik (tidak bergantung pada git) untuk ini.

Semua, v3.3.2 telah dirilis, ini mengunduh ketergantungan diagnostics bercabang dari NPM bukan GitHub sehingga ini akan bekerja untuk Anda. Silakan konfirmasi atau tolak. Terima kasih!

Terima kasih untuk perbaikannya.