Chore/update grpc identity

[zigfrei]

Критерии yarn check yarn test unit yarn workspace @atls/nestjs-grpc-identity build yarn workspace @atls/nestjs-grpc-identity prepack Проходит без ошибок

версия get-port "get-port": "6.1.2" https://github.com/atls/nestjs/issues/240

[zigfrei]

@SlumberyDude если в kratos, hydra, grpc-http-proxy убрать патч ошибка опять появляется. Может откатить патч и оставить обновление "jwks-rsa": до весрсии 2.1.4 а не до версии 3.0.1

[TorinAsakura]

Надо бы по этим патчам попросить ревью у @TorinAsakura .

Думаю, что он слишком разбросан по всем пакетам: hydra, kratos и др. Это точно не очень хорошо. Я бы наверное пробовал зафиксировать версию экспресса и/или его типов внутри identity на более раннюю версию. Или по крайней мере локализовал бы патч только для пакета identity и корневого модуля.

в чём вопрос?

[zigfrei]

@TorinAsakura допустимо ли что изменения из-за патча попали в зависимости других пакетов kratos, hydra, grpc-http-proxy? Если нет, то предлагаю откатить до совместимой версии "jwks-rsa": 2.1.4 и делать ПР.

[TorinAsakura]

@zigfrei Не понял, как это попали в зависимости других пакетов?

[zigfrei]

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

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

Решил сделать патч в модуле @types/express@npm:4.17.13 пропатчить "@types/express-serve-static-core": "^4.17.18" до версии "@types/express-serve-static-core": "^4.17.33"

Ошибки yarn check прошли но в зависимости kratos, hydra, grpc-http-proxy патч тоже добавился:

packages/hydra/package.json "devDependencies":{ "@types/express": "patch:@types/express@npm%3A4.[17.13#~/.yarn/patches/@types-express-npm-4.17.13-0e12fe9c24.patch](mailto:17.13#~/.yarn/patches/@types-express-npm-4.17.13-0e12fe9c24.patch)" }

`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'`

[TorinAsakura]

Нет, это ненормально, не нужно трогать эти зависимости, они могут сломать обратную совместимость

[zigfrei]

@TorinAsakura правильно ли будет откатить патч, обновление jwks-rsa поднять до версии 2.1.4 в которой yarn check не выдает ошибку и делать ПР?

[TorinAsakura]

Да

[SlumberyDude]

@zigfrei Можешь отменить реквест для @TorinAsakura? Я его сделаю