Skip to content

build(deps): bump aiohttp to 3.14.1 and cryptography to 48.0.1 - #3394

Merged
deacon-mp merged 3 commits into
masterfrom
build/bump-aiohttp-cryptography
Jul 8, 2026
Merged

build(deps): bump aiohttp to 3.14.1 and cryptography to 48.0.1#3394
deacon-mp merged 3 commits into
masterfrom
build/bump-aiohttp-cryptography

Conversation

@deacon-mp

Copy link
Copy Markdown
Contributor

Summary

Bumps aiohttp to 3.14.1 and cryptography to 48.0.1 in a single change. Dependabot opened these as two separate PRs (#3391, #3392), but neither clears the Safety gate on its own — each leaves the other package vulnerable, so pip-audit still reports findings and the build stays red. Bumping both together clears all of them. This supersedes #3391 and #3392.

Changes

  • requirements.txt: aiohttp 3.13.4 → 3.14.1 (clears 11 CVEs — CVE-2026-34993 / -47265 / -50269 / -54273–54280) and cryptography 46.0.7 → 48.0.1 (clears GHSA-537c-gmf6-5ccf).

Test plan

  • pip-audit -r requirements.txt — no known vulnerabilities.
  • Full tox suite (py3.10–3.13 + style + coverage + bandit + safety) passes.
  • Clean server.py --insecure --build boot — health reports 15 plugins, UI built, no traceback.

Rollback: revert this commit; the change is two version pins with no migration or state change.

Clears pip-audit findings on master: aiohttp 3.13.4 (11 CVEs, fixed in
3.14.x) and cryptography 46.0.7 (GHSA-537c-gmf6-5ccf, fixed in 48.0.1).
Target versions match the upstream Dependabot proposals
(dependabot/pip/aiohttp-3.14.1, dependabot/pip/cryptography-48.0.1).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


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

@deacon-mp
deacon-mp requested a review from Copilot June 23, 2026 17:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@deacon-mp deacon-mp self-assigned this Jul 8, 2026
@deacon-mp
deacon-mp requested a review from clenk July 8, 2026 16:50
clenk
clenk previously approved these changes Jul 8, 2026

@clenk clenk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pjfanning

Copy link
Copy Markdown
Member

@deacon-mp @clenk it appears that the broken CI jobs are what are causing all the merges to be blocked.

Your CI actions are out of date and many are not approved by the ASF.

I have already pushed a commit to this PR branch but there might need to be multiple iterations.

When this PR is merged, it should allow other PRs pass their security checks.

@pjfanning pjfanning left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm - needed to unblock other build issues

@pjfanning

Copy link
Copy Markdown
Member

@deacon-mp @clenk could you review this PR? I expect it to pass CI. Its progress seems good so far. Yet, I still get a status saying that Merging is blocked. You're not authorized to push to this branch

@deacon-mp
deacon-mp merged commit 28ce9ba into master Jul 8, 2026
9 checks passed
@deacon-mp
deacon-mp deleted the build/bump-aiohttp-cryptography branch July 8, 2026 23:21
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.

4 participants