Skip to content
This repository was archived by the owner on Oct 27, 2026. It is now read-only.

Fix open security vulnerabilities + gate PRs in CI - #57

Merged
dargmuesli merged 1 commit into
masterfrom
fix-vulnerabilities
Oct 26, 2026
Merged

dargmuesli merged 1 commit into
masterfrom
fix-vulnerabilities

Conversation

@dargmuesli

Copy link
Copy Markdown
Member

Fixes all open security alerts in this repo.

Dependency fixes:

  • mistune 2.0.0rc1 to 3.4.0 (6 high + 8 moderate advisories, fixed in 3.3.0). The v3 API removed the legacy Markdown(AstRenderer) class, so parse_protocol was migrated to the v3 token API: mistune.Markdown() with renderer=None returns the token tree, heading level moved to attrs.level, inline text nodes use raw. The existing test suite (test_post_protocol) covers this and passes.
  • pytest pin was >=8.3.3,<9, which blocked the fix for GHSA-6w46-j5rx-g56g (fixed in 9.0.3); relaxed to >=8.3.3, now on 9.1.1
  • pillow 11.0.0 to 13.0.0 (transitive via matplotlib; 3 critical + several high/moderate advisories)
  • jinja2 3.1.4 to 3.1.6, fonttools 4.54.1 to 4.67.0 (moderate)
  • matplotlib 3.9.2 to 3.11.2 (current 3.x, has wheels for 3.13)

CI:

  • the workflow previously only ran on push, so pull requests had no checks at all (always green). It now also runs on pull_request, so this and future PRs are gated by the existing tests, pre-commit and docker build jobs.

All 68 tests pass locally (uv sync and pytest).

- mistune 2.0.0rc1 to 3.4.0 (6 high + 8 moderate advisories): v3 removed the legacy Markdown(AstRenderer) API; parse_protocol now uses the v3 token API (mistune.Markdown() with renderer None, attrs.level, raw text)
- pytest pin relaxed from >=8.3.3,<9 to >=8.3.3 so 9.1.1 (fixes GHSA-6w46-j5rx-g56g) can be used
- pillow 11.0.0 to 13.0.0 (via transitive matplotlib constraint, 8 critical + high + moderate advisories)
- jinja2 3.1.4 to 3.1.6, fonttools 4.54.1 to 4.67.0 (moderate advisories)
- matplotlib 3.9.2 to 3.11.2 (current 3.x with wheels)
- CI: also run on pull_request, so PRs are actually gated (previously only push)

All 68 tests pass locally (uv sync + pytest).
@dargmuesli
dargmuesli requested review from nikeee and soerface October 26, 2026 21:10
@dargmuesli
dargmuesli merged commit 067d60d into master Oct 26, 2026
9 checks passed
@dargmuesli
dargmuesli deleted the fix-vulnerabilities branch October 26, 2026 21:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants