atls / nestjs

NestJS Custom Workshop
BSD 3-Clause "New" or "Revised" License
33 stars 8 forks source link

Обновление `grpc-identity` #240

Closed sollidy closed 1 year ago

sollidy commented 1 year ago

С чем связан запрос на фичу?

Обновление зависимостей в репозитории grpc-identity

Расскажите как вы это себе видите

Необходимо обновить все зависимости пакета. @nestjs/... выше мажорной 8 версии не ставить.

Критерий успеха

Материалы

Интерактивный апгрейд через yarn:

zigfrei commented 1 year ago

Все зависимости кроме jwks-rsa и get-port обновились до последних.

При обновлении jwks-rsa выше версии 2.1.4 yarn checkвыдает ошибку

@SlumberyDude предлагаю поднять jwks-rsa на версию 2.1.4 и делать ПР

`Erro: Conflicting definitions for 'express-serve-static-core' found at '/home/zigfrei/.yarn/berry/cache/@types-express-serve-static-core-npm-4.17.36-87a82f5a6b-9.zip/node_modules/@types/express-serve-static-core/index.d.ts' and '/home/zigfrei/.yarn/berry/cache/@types-express-serve-static-core-npm-4.17.24-cd6b9fd3ba-9.zip/node_modules/@types/express-serve-static-core/index.d.ts'.` `Consider installing a specific version of this library to resolve the conflict. The file is in the program because: Type library referenced via 'express-serve-static-core' from file '/home/zigfrei/.yarn/berry/cache/@types-express-npm-4.17.13-0e12fe9c24-9.zip/node_modules/@types/express/index.d.ts' with packageId '@types/express-serve-static-core/index.[d.ts@4.17.24](mailto:d.ts@4.17.24)' Imported via 'express-serve-static-core' from file '/home/zigfrei/.yarn/berry/cache/@types-express-npm-4.17.13-0e12fe9c24-9.zip/node_modules/@types/express/index.d.ts' with packageId '@types/express-serve-static-core/index.[d.ts@4.17.24](mailto:d.ts@4.17.24)' Type library referenced via 'express-serve-static-core' from file '/home/zigfrei/.yarn/berry/cache/@types-express-npm-4.17.17-46fe8173db-9.zip/node_modules/@types/express/index.d.ts' with packageId '@types/express-serve-static-core/index.d.ts@4.17.36'` @SlumberyDude предлагаю поднять `jwks-rsa` на версию 2.1.4 и делать ПР
SlumberyDude commented 1 year ago

@zigfrei Ошибка связана с экспрессом и его типами. У меня была такая в http-proxy модуле, поправил снижением версий, можешь попробовать.

Еще длинные логи заворачивай в тег <details> </deatils>

Вот такой
zigfrei commented 1 year ago

@SlumberyDude изначально "jwks-rsa": "^2.0.5" максимальная версия до ошибки экспресса и его типов "jwks-rsa": "2.1.4"

поправил снижением версий, можешь попробовать.

Снижением версии jwks-rsa?

SlumberyDude commented 1 year ago

@zigfrei

Снижением версии jwks-rsa?

Не, я снижал экспресс вроде, там не было jwks-rsa. Еще странно что обновление jwks-rsa дает ошибку по типам в экспрессе. Еще нигде в путях в логах jwks-rsa не фигурирует. Попробуй разобраться в чем тут дело.

У тебя конфликтуют 2 модуля @types-express-serve-static-core в дереве зависимостей потому что один из них 4.17.36 а другой 4.17.24. Первый является зависимостью @types-express-npm-4.17.17 а другой @types-express-npm-4.17.13. Если привести их к одному патчу, то может удастся пофиксить.

zigfrei commented 1 year ago

В модуле @types/express@npm:4.17.13 пропатчил "@types/express-serve-static-core": "^4.17.18" до версии "@types/express-serve-static-core": "^4.17.33" после чего yarn check прошел без ошибок