Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): browsers and included rebuild on factory:4.2.1 #1221

Closed
wants to merge 1 commit into from

Conversation

MikeMcC399
Copy link
Collaborator

Issue

Concerning

Image Debian Published Version
cypress/browsers 12.7 Sep 25, 2024 node-20.17.0-chrome-129.0.6668.70-1-ff-130.0.1-edge-129.0.2792.52-1
cypress/included 12.7 Sep 25, 2024 13.15.0
trivy image --ignore-unfixed --pkg-types os --scanners vuln --severity CRITICAL cypress/browsers:latest
trivy image --ignore-unfixed --pkg-types os --scanners vuln --severity CRITICAL cypress/included:latest

reports critical fixed issues not yet installed

cypress/browsers:latest (debian 12.7)

Total: 5 (CRITICAL: 5)

┌───────────┬────────────────┬──────────┬────────┬───────────────────┬────────────────────┬─────────────────────────────────────────────────────────────┐
│  Library  │ Vulnerability  │ Severity │ Status │ Installed Version │   Fixed Version    │                            Title                            │
├───────────┼────────────────┼──────────┼────────┼───────────────────┼────────────────────┼─────────────────────────────────────────────────────────────┤
│ git       │ CVE-2024-32002 │ CRITICAL │ fixed  │ 1:2.39.2-1.1      │ 1:2.39.5-0+deb12u1 │ git: Recursive clones RCE                                   │
│           │                │          │        │                   │                    │ https://avd.aquasec.com/nvd/cve-2024-32002                  │
├───────────┤                │          │        │                   │                    │                                                             │
│ git-man   │                │          │        │                   │                    │                                                             │
│           │                │          │        │                   │                    │                                                             │
├───────────┼────────────────┤          │        ├───────────────────┼────────────────────┼─────────────────────────────────────────────────────────────┤
│ libexpat1 │ CVE-2024-45490 │          │        │ 2.5.0-1           │ 2.5.0-1+deb12u1    │ libexpat: Negative Length Parsing Vulnerability in libexpat │
│           │                │          │        │                   │                    │ https://avd.aquasec.com/nvd/cve-2024-45490                  │
│           ├────────────────┤          │        │                   │                    ├─────────────────────────────────────────────────────────────┤
│           │ CVE-2024-45491 │          │        │                   │                    │ libexpat: Integer Overflow or Wraparound                    │
│           │                │          │        │                   │                    │ https://avd.aquasec.com/nvd/cve-2024-45491                  │
│           ├────────────────┤          │        │                   │                    ├─────────────────────────────────────────────────────────────┤
│           │ CVE-2024-45492 │          │        │                   │                    │ libexpat: integer overflow                                  │
│           │                │          │        │                   │                    │ https://avd.aquasec.com/nvd/cve-2024-45492                  │
└───────────┴────────────────┴──────────┴────────┴───────────────────┴────────────────────┴─────────────────────────────────────────────────────────────┘
cypress/included:latest (debian 12.7)

Total: 5 (CRITICAL: 5)

...
(as above for cypress/browsers:latest)

Change

In factory/.env, bump environment variable

to rebuild cypress/browsers and cypress/included based on cypress/factory:4.2.1, which includes Debian 12.x published fixes from the Debian repository.

Verify

cd factory
docker pull cypress/factory:4.2.1
docker compose build browsers
trivy image --ignore-unfixed --pkg-types os --scanners vuln --severity CRITICAL cypress/browsers
docker compose build included
trivy image --ignore-unfixed --pkg-types os --scanners vuln --severity CRITICAL cypress/included:latest

should show

cypress/browsers (debian 12.7)

Total: 0 (CRITICAL: 0)

cypress/included (debian 12.7)

Total: 0 (CRITICAL: 0)

@cypress-app-bot
Copy link

@MikeMcC399
Copy link
Collaborator Author

An alternative would be to wait until there is a new version of Node.js and bump both Node.js and at least one browser version in a single PR. This would trigger new builds of cypress/base, cypress/browsers and cypress/included in one PR.

According to https://github.com/nodejs/node/pulls?q=is%3Apr+is%3Aopen+release+in%3Atitle there is a new release of Node.js 20.x planned later this week.

I leave it to the Cypress.io team to decide whether to wait or whether to go ahead with this PR.

@MikeMcC399 MikeMcC399 marked this pull request as draft October 1, 2024 13:28
@MikeMcC399
Copy link
Collaborator Author

MikeMcC399 commented Oct 1, 2024

Firefox 131.0 was released today, Oct 1, 2024, so I would favor delaying and re-submitting the PR to include more updates.

@MikeMcC399 MikeMcC399 closed this Oct 1, 2024
@MikeMcC399 MikeMcC399 deleted the update/edge branch October 1, 2024 13:34
@MikeMcC399 MikeMcC399 removed the request for review from jennifer-shehane October 1, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants