Skip to content

Pin nokogiri >= 1.19.3 (GHSA-c4rq-3m3g-8wgx)#1807

Merged
mokagio merged 2 commits into
trunkfrom
mokagio/nokogiri-1.19.3
May 12, 2026
Merged

Pin nokogiri >= 1.19.3 (GHSA-c4rq-3m3g-8wgx)#1807
mokagio merged 2 commits into
trunkfrom
mokagio/nokogiri-1.19.3

Conversation

@mokagio
Copy link
Copy Markdown
Contributor

@mokagio mokagio commented May 7, 2026

Summary

Adds an explicit gem 'nokogiri', '>= 1.19.3' pin to Gemfile to pull in the fix for GHSA-c4rq-3m3g-8wgx — a high-severity ReDoS in Nokogiri's CSS selector tokenizer (vulnerable < 1.19.3).

This repo is on fastlane-plugin-wpmreleasetoolkit ~> 13.8, which predates the toolkit's own nokogiri >= 1.19.3 floor (added in 14.4.1). Pinning explicitly here closes the gap without requiring a release-toolkit major bump.

Generated as part of the nokogiri 1.19.3 Orchard campaign covering all release-toolkit consumers.

Testing

bundle install. Gemfile.lock resolves nokogiri to 1.19.3.


Posted by Claude Code (Opus 4.7) on behalf of @mokagio with approval.

Carries the fix for [GHSA-c4rq-3m3g-8wgx][advisory] (high-severity ReDoS
in Nokogiri's CSS selector tokenizer; vulnerable `< 1.19.3`).

This repo is on `fastlane-plugin-wpmreleasetoolkit ~> 13.8`, which
predates the toolkit's own `nokogiri >= 1.19.3` floor (added in 14.4.1).
Pinning explicitly here closes the gap until a future toolkit-major bump
makes the pin redundant.

[advisory]: GHSA-c4rq-3m3g-8wgx

---

Generated with the help of Claude Code, https://claude.com/claude-code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 7, 2026 12:01
@mokagio mokagio added this to the Future milestone May 7, 2026
@mokagio mokagio added the tooling Related to anything that supports the building & maintaining of the project. label May 7, 2026
@mokagio mokagio self-assigned this May 7, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR mitigates GHSA-c4rq-3m3g-8wgx by explicitly pinning Nokogiri to a non-vulnerable version in this repo’s Bundler setup, independent of the older fastlane-plugin-wpmreleasetoolkit constraint.

Changes:

  • Add an explicit nokogiri >= 1.19.3 dependency to the Gemfile.
  • Update Gemfile.lock to resolve Nokogiri to 1.19.3 and record the new direct dependency.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
Gemfile Adds an explicit Nokogiri version floor to ensure the security fix is pulled in.
Gemfile.lock Locks Nokogiri to 1.19.3 and includes Nokogiri in direct dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Gemfile Outdated
@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented May 7, 2026

📲 You can test the changes from this Pull Request in Simplenote Android by scanning the QR code below to install the corresponding build.

App NameSimplenote Android
Build TypeDebug
Commit3391d90
Direct Downloadsimplenote-android-prototype-build-pr1807-3391d90-019e1d18-3ddf-45b0-a85c-8e9ddcee5fad.apk

@mokagio mokagio requested review from iangmaia and twstokes May 7, 2026 12:07
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mokagio mokagio enabled auto-merge May 8, 2026 04:55
@mokagio
Copy link
Copy Markdown
Contributor Author

mokagio commented May 12, 2026

Mmm... what's up with CI I wonder

image

@mokagio mokagio merged commit abf606f into trunk May 12, 2026
11 of 13 checks passed
@mokagio mokagio deleted the mokagio/nokogiri-1.19.3 branch May 12, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling Related to anything that supports the building & maintaining of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants