Skip to content

Upgrade to NextJS 11 to fix CVEs#171

Merged
EnriqueL8 merged 4 commits into
hyperledger:mainfrom
SamMayWork:fix-cves
Dec 11, 2025
Merged

Upgrade to NextJS 11 to fix CVEs#171
EnriqueL8 merged 4 commits into
hyperledger:mainfrom
SamMayWork:fix-cves

Conversation

@SamMayWork

Copy link
Copy Markdown
Contributor

ref: hyperledger/firefly-tokens-erc20-erc721#189

Similar to what we did for the erc20/erc721 token connector, this PR suggests bumping to NestJS 11 to resolve a number of CVEs in the application.


For more information on the migrations and features of NestJS 11 see:

Buuuuut at a high-level, NestJS 11 brings a newer version of express/fastify and with it some changes to how wildcard routes are defined w/ some breaking changes to niche query features that we are not using within this application.


...and for sake of completeness, the full list of CVEs that this PR addresses is:

Signed-off-by: SamMayWork <sam.may@kaleido.io>
@SamMayWork SamMayWork requested a review from a team as a code owner December 11, 2025 15:08
Signed-off-by: SamMayWork <sam.may@kaleido.io>
Signed-off-by: SamMayWork <sam.may@kaleido.io>
Signed-off-by: SamMayWork <sam.may@kaleido.io>
@SamMayWork

Copy link
Copy Markdown
Contributor Author

Noting that the Docker build is failing due to issues w/ Trivy download which is strange considering the ERC20 PR is just fine. I am noticing a large number of 500's coming from GitHub so wondering if there are other issues going on.

@EnriqueL8

Copy link
Copy Markdown
Contributor

Have re-run the jobs

@EnriqueL8 EnriqueL8 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the great work here @SamMayWork - testing and explaining why the Nest 11 upgrade is safe and fixes loads of CVEs 🙇🏼

See the explanation here hyperledger/firefly-tokens-erc20-erc721#189 (comment)

@EnriqueL8 EnriqueL8 merged commit 473e496 into hyperledger:main Dec 11, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants