Skip to content

Fix Chrome & Chromedriver installation using Chrome for Testing#14

Merged
rayunzer merged 4 commits intomasterfrom
resolve-chrome-driver-to-major-version
Mar 25, 2026
Merged

Fix Chrome & Chromedriver installation using Chrome for Testing#14
rayunzer merged 4 commits intomasterfrom
resolve-chrome-driver-to-major-version

Conversation

@rayunzer
Copy link
Copy Markdown
Contributor

@rayunzer rayunzer commented Mar 24, 2026

Problem

Chrome for Testing does not publish a chromedriver binary for every Chrome release. When the installed Chrome version (e.g. 146.0.7680.164) has no matching chromedriver at the exact URL, the wget command fails and the action errors out.
https://github.com/QuickPay/manager/actions/runs/23494135067/job/68371526465

Solution

Replace the old approach with a single fetch from the Chrome for Testing API:

Fix chromedriver download fallback when exact version is unavailable
@rayunzer rayunzer force-pushed the resolve-chrome-driver-to-major-version branch from 83ff638 to e48a490 Compare March 24, 2026 15:15
@rayunzer rayunzer force-pushed the resolve-chrome-driver-to-major-version branch from 22a3d08 to da355cb Compare March 24, 2026 15:25
@rayunzer rayunzer marked this pull request as ready for review March 24, 2026 15:27
@rayunzer rayunzer requested a review from a team March 24, 2026 15:29
@rayunzer rayunzer force-pushed the resolve-chrome-driver-to-major-version branch from 7d83481 to 4a277c0 Compare March 25, 2026 08:43
@rayunzer rayunzer requested a review from gnulinooks March 25, 2026 08:44
@rayunzer rayunzer changed the title Fix chromedriver download with fallback to nearest known-good build Fix Chrome & Chromedriver installation using Chrome for Testing Mar 25, 2026
Copy link
Copy Markdown

@gnulinooks gnulinooks left a comment

Choose a reason for hiding this comment

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

Looks good.

@rayunzer rayunzer merged commit 300ac3a into master Mar 25, 2026
5 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