Skip to content

Update brotli to 1.2.1 (PYSEC-2026-4322) - #247

Merged
ChristianHandy merged 1 commit into
mainfrom
fix-brotli-pysec-2026-4322
Oct 23, 2026
Merged

ChristianHandy merged 1 commit into
mainfrom
fix-brotli-pysec-2026-4322

Conversation

@ChristianHandy

Copy link
Copy Markdown
Owner

Summary

pip-audit ("Check dependencies for known vulnerabilities") fails the pytest jobs on every pull request (e.g. #246): brotli 1.2.0 has PYSEC-2026-4322, fixed in 1.2.1.

Changes

  • requirements-lock.txt: brotli 1.2.0 → 1.2.1, regenerated with uv pip compile ... --upgrade-package brotli (no other package changes; the dev lock has no brotli pin of its own).
  • CHANGELOG.md: entry under Unreleased → Security.

Verification

  • pip-audit --require-hashes --disable-pip -r requirements-lock.txt → no known vulnerabilities
  • same for requirements-dev-lock.txt → no known vulnerabilities

AI assistance

Prepared with Claude Code.

🤖 Generated with Claude Code

pip-audit fails the pytest jobs on every pull request because brotli
1.2.0 has a known vulnerability. Only brotli changes in the lock file.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: ChristianHandy <christianhandy180@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
pip/brotli 1.2.1 🟢 7.6
Details
CheckScoreReason
Code-Review⚠️ 1Found 4/21 approved changesets -- score normalized to 1
Packaging⚠️ -1packaging workflow not detected
Security-Policy🟢 10security policy file detected
Maintained🟢 1030 commit(s) and 4 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow⚠️ -1no workflows found
Token-Permissions⚠️ -1No tokens found
Pinned-Dependencies⚠️ -1no dependencies found
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 2badge detected: InProgress
Vulnerabilities🟢 100 existing vulnerabilities detected
Dependency-Update-Tool🟢 10update tool detected
Fuzzing🟢 10project is fuzzed
SAST🟢 9SAST tool is not run on all commits -- score normalized to 9
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
License🟢 10license file detected
CI-Tests🟢 1022 out of 22 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 16 contributing companies or organizations

Scanned Files

  • requirements-lock.txt

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

🟢 Ready to merge

  • All required checks passed and there are no merge conflicts. A maintainer still reviews the change before merging.

Commit 53edbcf; updated whenever a check finishes.

What this pull request changes

2 files, +145 / -101 lines.

Area Files Added Deleted
Dependencies 1 +144 -101
Documentation 1 +1 -0
Changed files
  • CHANGELOG.md (modified, +1 / -0)
  • requirements-lock.txt (modified, +144 / -101)

Required checks

Check Result
pytest (Python 3.11) ✅ passed
pytest (Python 3.12) ✅ passed
pytest (Python 3.13) ✅ passed
Analyze (python) ✅ passed
Safe to merge ✅ passed
AI assistance ✅ passed

Other checks: 12 passed, 0 failed, 0 running.

@ChristianHandy

Copy link
Copy Markdown
Owner Author

/allow-ai 53edbcf brotli security update

@ChristianHandy
ChristianHandy merged commit b32b7c8 into main Oct 23, 2026
25 of 26 checks passed
@ChristianHandy
ChristianHandy deleted the fix-brotli-pysec-2026-4322 branch October 23, 2026 16:00
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.

1 participant