Skip to content

fix(deps): bump pygments to >=2.20.0 for ReDoS CVE#478

Open
sergioestebance wants to merge 1 commit into
mainfrom
fix/bump-pygments-redos-cve
Open

fix(deps): bump pygments to >=2.20.0 for ReDoS CVE#478
sergioestebance wants to merge 1 commit into
mainfrom
fix/bump-pygments-redos-cve

Conversation

@sergioestebance

Copy link
Copy Markdown
Contributor

Summary

Production impact

None. Pygments is a transitive dependency pulled in by Rich and pytest. Production code only uses Rich's console/spinner/text features — no syntax highlighting or Pygments lexers are invoked. No untrusted input flows through Pygments' regex engine.

Test plan

  • uv lock resolves pygments to 2.20.0
  • No changes to direct dependencies

@github-actions github-actions Bot added the low-risk-change PR qualifies as low-risk per policy and can be merged without manual review label May 18, 2026
@sergioestebance sergioestebance force-pushed the fix/bump-pygments-redos-cve branch from f0b3023 to c74d9dc Compare May 18, 2026 12:24
@github-actions github-actions Bot added low-risk-change PR qualifies as low-risk per policy and can be merged without manual review and removed low-risk-change PR qualifies as low-risk per policy and can be merged without manual review labels May 18, 2026
@sergioestebance sergioestebance force-pushed the fix/bump-pygments-redos-cve branch from c74d9dc to 558de6b Compare May 18, 2026 12:51
@github-actions github-actions Bot removed the low-risk-change PR qualifies as low-risk per policy and can be merged without manual review label May 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Automated low-risk assessment

This PR was evaluated against the repository's Low-Risk Pull Requests procedure and does not qualify as low risk.

The PR modifies dependency constraints and the lockfile to bump the third‑party package Pygments to 2.20.0 to address a ReDoS CVE. Changes to dependency versions/lockfiles affect external third‑party code and runtime behavior and are not covered by the allowed low‑risk categories, so this should follow the normal review process despite the author’s note about limited production impact.

This PR requires a manual review before merging.

@sergioestebance sergioestebance self-assigned this May 18, 2026
@sergioestebance sergioestebance added the dependencies Pull requests that update a dependency file label May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant