Fix/dependency vulnerability #22
Conversation
Isabelar07
requested review from
caio-emidio,
RodrigoDornelles,
vcwild and
DevRadhy
|
Resolves #21 |
There was a problem hiding this comment.
Eu não manjo tanto do eco sistema de javascript, mas apareceu alguns alertas de dependências ultrapassadas, consegue aferir o impacto e talvez corrigir?
npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated [email protected]: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated [email protected]: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
npm WARN deprecated [email protected]: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
npm WARN deprecated [email protected]: some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features added
npm WARN deprecated @types/[email protected]: Mongoose publishes its own types, so you do not need to install this package.
|
não estamos usando uuid, usamos o object_id do mongo. Não sei se object_id usa o uuid por debaixo dos panos, mas acredito que isso não vai impactar na api, assim como os demais. Mas posso "atualizar" se for o caso. Em relação ao type do mongoose, vou deixar com ele mesmo. O emídio tem mais experiência com mongo e nestjs do que eu, então acredito que ele colocou o type por algum motivo. |
There was a problem hiding this comment.
Aprovado então! deixo para o @caio-emidio considerar essas coisas futuramente!
|
Um esquema ali era ter dado upgrade e ver se vai dar B.O. |
fixes vulnerability of dependencies