Skip to content

Conversation

nileshgulia1
Copy link
Member

@nileshgulia1 nileshgulia1 commented Oct 2, 2025

execa <2.0.0 has a critical security vulnerability in docker. According to docker scout:

   1C     0H     0M     0L  execa 0.6.3
pkg:npm/[email protected]

Dockerfile (22:22)
COPY --from=builder /app/ /app/

    ✗ CRITICAL GMS-2020-2 [OWASP Top Ten 2017 Category A9 - Using Components with Known Vulnerabilities]
      https://scout.docker.com/v/GMS-2020-2
      Affected range : <2.0.0                                        
      Fixed version  : 2.0.0                                         
      CVSS Score     : 9.8                                           
      CVSS Vector    : CVSS:3.0/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H  
      

I updated it to the latest version with a small API change. Right now, execa 1.0.0 is still being installed through @testing-library/jest-dom and razzle (which isn’t maintained anymore). [email protected] is also a direct dependency of "sane" (v4.x, which again comes from majorly "jest"). This issue is fixed in [email protected], so it’s worth considering an upgrade/override.

"chalk": "4",
"commander": "8.2.0",
"comment-json": "^4.2.3",
"execa": "0.6.3",
Copy link
Member Author

Choose a reason for hiding this comment

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

@sneridagh Any idea why we are using very old version, bundlesize?

Copy link
Member

@wesleybl wesleybl left a comment

Choose a reason for hiding this comment

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

LGTM

@wesleybl
Copy link
Member

wesleybl commented Oct 3, 2025

Right now, execa 1.0.0 is still being installed through @testing-library/jest-dom and razzle (which isn’t maintained anymore). [email protected] is also a direct dependency of "sane" (v4.x, which again comes from majorly "jest"). This issue is fixed in [email protected], so it’s worth considering an upgrade/override.

I agree with that.

In fact, I'm wondering if it's worth creating a new "@volto/razzle" package to update these packages. But maybe that wouldn't be worth it because these issues will be fixed in Seven.

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