Repository navigation
fix: Update brotli version to 1.2.1 for security improvements - #1297
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
| Layer / File(s) | Summary |
|---|---|
Brotli requirement and pin performance-test/requirements.in, performance-test/requirements.txt |
The input requirements add brotli>=1.2.1. The pinned requirements update Brotli to 1.2.1 and list requirements.in alongside geventhttpclient. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Suggested reviewers: cpcundill
Merge Risk: 
Minimal · previous review 83cfc
-
The dependency update is ready to merge with normal installation checks.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
A rabbit checks the version line,
Brotli’s pin is now in line.
Requirements list the source,
Both files follow the same course.
The rabbit hops away, all fine.
Comment @coderabbitai help to get the list of available commands.
Coverage Report
File CoverageNo changed files found. |
What type of PR is this? (check all applicable)
Description
Updates Brotli from 1.1.0 to 1.2.1 in the performance-test dependencies to address CVE-2026-111497.
Adds
brotli>=1.2.1torequirements.inand regenerates the lockfile.Related Tickets & Documents
QA Instructions, Screenshots, Recordings
Validated in an isolated Python 3.10 container:
No UI changes; browser testing is not applicable.
Added/updated tests?
[optional] Are there any post deployment tasks we need to perform?
Reinstall dependencies in existing performance-test environments. Rebuild and rescan any images containing this dependency set.
[optional] Are there any dependencies on other PRs or Work?
None.
Summary by CodeRabbit