Замена yauzl на adm-zip
Closed this issue · 3 comments
Судя по активности в репозитории yauzl разработка данного пакета заброшена.
Также на данный момент есть один неприятный warning связанный с DEP0005
Подробнее здесь: thejoshwolfe/yauzl#115
Как можно видеть здесь: https://github.com/AuroraTeam/LauncherServer/blob/86b65324a00e71ef6642ad393ac36fba2fcc906e/package.json#L62 у нас используется форк с исправлением из того же пулл реквеста, указанного выше.
Как предложение заменить yauzl на adm-zip.
Он поддерживается разработчиками. И вроде более менее совпадает по функционалу с yauzl.
Мне ранее предлагал @CKATEPTb, но я пока откладывал этот вопрос.
Если судить по npm-trends https://www.npmtrends.com/adm-zip-vs-yauzl-vs-extract-zip у yauzl больше скачиваний, но как можно заметить большую часть скачиваний приносит пакет extract-zip (он является обёрткой над yauzl).
Единственное нужно будет проверить что adm-zip:
- Умеет читать jar файлы
- Работает также или даже быстрее
Ссылки:
yauzl: https://github.com/thejoshwolfe/yauzl
adm-zip: https://github.com/cthackers/adm-zip
Временно закрываю ишью
yauzl ~ 24сек (на примере ассетов 1.14.4)
adm-zip ~ 3.5сек. по скорости очевидно вин
