Skip to content
New issue

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

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

Already on GitHub? # to your account

Pin cheerio to avoid the breaking change of undici #5142

Merged
merged 1 commit into from
Sep 29, 2024
Merged

Conversation

louislam
Copy link
Owner

@louislam louislam commented Sep 29, 2024

Description

Fixes #5141

cheerio 1.0.0 is depending on new undici which is not supported on Node.js 16 (Alpine), but it is working on Node.js 18 (Debian).

Pinning cheerio to 1.0.0-rc.12 could solve the issue.

The auto test actually showed the error, I don't why I ignored it, lol.

https://github.com/louislam/uptime-kuma/actions/runs/11076765407/job/30780591154

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@louislam louislam merged commit c6d6061 into 1.23.X Sep 29, 2024
14 checks passed
@CommanderStorm CommanderStorm deleted the 1.23.X-fix branch September 30, 2024 02:21
@CommanderStorm CommanderStorm restored the 1.23.X-fix branch September 30, 2024 02:21
@CommanderStorm CommanderStorm deleted the 1.23.X-fix branch September 30, 2024 02:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant