fix(deps): resolve dependabot security alerts in integration lockfiles - #803
Merged
Abhijeet Prasad (AbhiPrasad) merged 1 commit intoSep 24, 2026
Merged
Conversation
Targeted security bumps in the two deprecated integration compatibility packages, whose lockfiles had drifted behind `py/uv.lock`: integrations/adk-py/uv.lock anyio 4.13.0 -> 4.14.2 cryptography 48.0.1 -> 50.0.1 idna 3.13 -> 3.20 pyopenssl 26.2.0 -> 26.4.0 (required by cryptography 50) integrations/langchain-py/uv.lock anyio 4.13.0 -> 4.14.2 cryptography 49.0.0 -> 50.0.1 idna 3.13 -> 3.20 Advisories cleared: GHSA-82r6-8w77-94w6 CRITICAL anyio: TLSStream IDNA 2003 host name encoding enables TLS certificate spoofing GHSA-5p39-cfhj-2xmp MODERATE anyio: process-pool workers can block indefinitely on undrained stderr GHSA-g6cj-pr64-35w5 HIGH cryptography: PKCS#7 EnvelopedData Bleichenbacher oracle GHSA-jwv3-5hgf-82ww HIGH cryptography: duplicate self-signed intermediates cause exponential path-building GHSA-m2h6-j472-rp4c MODERATE cryptography: verifier accepts wildcard DNS names, escaping permittedSubtrees GHSA-65pc-fj4g-8rjx MODERATE idna: crafted input to idna.encode() bypasses the CVE-2024-3651 fix Both lockfiles now scan clean against OSV. Validated by running each package's full CI step list (uv sync, ruff check, pytest, import smoke test); all green. Not fixed, because no patched release exists upstream: diskcache 5.6.3 (GHSA-w8v5-vhqr-4h9v) and nltk 3.10.3 (GHSA-8mgp-746c-j5xp) in py/uv.lock. Both are transitive dev-only deps of the `lint` group (via dspy and llama-index-core); neither is a runtime dependency of the published braintrust package. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Abhijeet Prasad (AbhiPrasad)
deleted the
fix/dependabot-security-alerts
branch
September 24, 2026 21:06
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Clears six open Dependabot security advisories by bumping
anyio,cryptographyandidnain the two deprecated integration compatibility packages. Both lockfiles had drifted behindpy/uv.lock, which already carried patched versions of all three.integrations/adk-pyintegrations/langchain-pyanyiocryptographyidnapyopensslpyopensslmoves only becausecryptography50 requires it.Advisories cleared
anyioTLSStreamIDNA 2003 host name encoding enables TLS certificate spoofingcryptographyEnvelopedDatadecryption exposes a Bleichenbacher oraclecryptographyanyiocryptographypermittedSubtreesidnaidna.encode()bypasses the CVE-2024-3651 fixApproach
Used targeted
uv lock --upgrade-package anyio --upgrade-package cryptography --upgrade-package idnain each package directory rather than a full re-lock, so the diff is limited to the four packages above.Validation
Re-audited both lockfiles against OSV after the bumps — both scan clean, 0 known vulnerabilities.
Ran the exact CI step list from
adk-py-test.yamlandlangchain-py-test.yamlfor each package (uv sync,ruff check,pytest, import smoke test). All green:integrations/adk-py— ruff clean,src/tests/test_reexports.py2 passed,braintrust_adk/setup_braintrustimport OKintegrations/langchain-py— ruff clean,src/tests2 passed,braintrust_langchain/BraintrustCallbackHandlerimport OKNot fixed — no patched release exists upstream
Two advisories in
py/uv.lockhave no fixed version available, so there is nothing to bump to:diskcache5.6.3 — GHSA-w8v5-vhqr-4h9v (Moderate), unsafe pickle deserialization. 5.6.3 is the latest release on PyPI, from 2023.nltk3.10.3 — GHSA-8mgp-746c-j5xp (High), model-artifact APIs bypass path security. 3.10.3 is the latest release and is markedlast_affected.Both are transitive dev-only dependencies of the
lintdependency group (viadspyandllama-index-corerespectively). Neither is a runtime dependency of the publishedbraintrustpackage.Options, if you want them off the alert list: dismiss them in the Dependabot UI as "no fix available", or drop
dspy/llama-index-corefrom the sharedlintgroup the waycrewaialready is (see the existing precedent comment inpy/pyproject.toml). Left alone here to avoid changing lint coverage unprompted.Note on how these were found
The
ghtoken available in this environment lacksvulnerability_alerts=read, so the alert list could not be read directly — the REST endpoint returns 403, and the GraphQLvulnerabilityAlertsfallback silently returnstotalCount: 0rather than erroring. The findings above were reproduced by parsing all three tracked lockfiles and querying the OSV API, which mirrors the same GHSA identifiers Dependabot uses.The push that created this branch surfaced GitHub's own count for the default branch: 4 vulnerabilities (2 critical, 2 moderate). That matches
anyio4.13.0 exactly — it carries one critical (GHSA-82r6-8w77-94w6) and one moderate (GHSA-5p39-cfhj-2xmp) advisory, and it is pinned at 4.13.0 in both integration lockfiles: 2 x critical + 2 x moderate = 4. On that reading, this PR clears all four open alerts.The
cryptographyandidnabumps are then additional hardening that OSV flags but Dependabot is not currently alerting on. They cost nothing here, since both packages had already advanced past these advisories inpy/uv.lock.Since the alert list could not be read directly, please sanity-check the above against the real Dependabot page when reviewing.
🤖 Generated with Claude Code
Co-authored by StarfolkAI (@starfolkai)[bot]