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

feat: download chrome-headless-shell by default and use it for the old headless mode #11754

Merged
merged 4 commits into from Jan 25, 2024

Conversation

OrKoN
Copy link
Collaborator

@OrKoN OrKoN commented Jan 25, 2024

With this PR, Puppeteer will download two binaries by default: a Chrome for Testing binary and a chrome-headless-shell of the same version. chrome-headless-shell will be used as a replacement for the old headless mode.

Two new configurations are added:

  1. PUPPETEER_SKIP_CHROME_DOWNLOAD to skip Chrome binary downloads (only chrome-headless-shell will be downloaded).
  2. PUPPETEER_SKIP_CHROME_HEADLESS_SHELL_DOWNLOAD to skip chrome-headless-shell binary downloads (only Chrome for Testing will be downloaded).

Use these variables if you only want to use the old headless mode (chrome-headless-shell) or only the new-headless/headful mode (Chrome for Testing). The existing configuration PUPPETEER_SKIP_DOWNLOAD would still apply to both binaries.

@OrKoN OrKoN force-pushed the orkon/chrome-headless-shell branch 2 times, most recently from 2d65e03 to 4d96cae Compare January 25, 2024 13:28
@OrKoN OrKoN marked this pull request as ready for review January 25, 2024 13:29
@OrKoN
Copy link
Collaborator Author

OrKoN commented Jan 25, 2024

Hey folks, this is a reland for #11093

@OrKoN OrKoN force-pushed the orkon/chrome-headless-shell branch from 4d96cae to 3bfd84f Compare January 25, 2024 13:53
@OrKoN OrKoN force-pushed the orkon/chrome-headless-shell branch 2 times, most recently from 0eeed62 to 29bbab9 Compare January 25, 2024 18:25
Copy link
Collaborator

@Lightning00Blade Lightning00Blade left a comment

Choose a reason for hiding this comment

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

LGTM, after my suggestion is accepted.

packages/puppeteer/src/node/install.ts Outdated Show resolved Hide resolved
@OrKoN OrKoN force-pushed the orkon/chrome-headless-shell branch from 29bbab9 to a040b63 Compare January 25, 2024 18:39
@OrKoN OrKoN merged commit ce894a2 into main Jan 25, 2024
38 checks passed
@OrKoN OrKoN deleted the orkon/chrome-headless-shell branch January 25, 2024 20:39
@release-please release-please bot mentioned this pull request Jan 25, 2024
romaricpascal added a commit to alphagov/govuk-frontend that referenced this pull request Jan 29, 2024
[Puppeteer now automatically downloads chrome-headless-shell](puppeteer/puppeteer#11754).
It's used for old headless mode, which we don't use so we don't need to download it.
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.

None yet

2 participants