Skip to content

[CD-1788] Replace critical compromised certifi CA bundle - #147

Closed
andrazk wants to merge 1 commit into
mainfrom
cd-1788-certifi-security
Closed

andrazk wants to merge 1 commit into
mainfrom
cd-1788-certifi-security

Conversation

@andrazk

@andrazk andrazk commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace certifi 2026.1.4 with 2026.10.19 to address critical GHSA-6q48-6c7w-9gw2 (compromised emSign intermediate keys in the old bundled trust roots).

Tracking: https://linear.app/casemarkai/issue/CD-1788
Advisory: GHSA-6q48-6c7w-9gw2
Vanta: https://app.vanta.com/c/casemark/tests/packages-checked-for-vulnerabilities-v2-records-closed-github-dependabot-critical

Scope

  • Only certifi changes in uv.lock; regenerate the existing requirements-dev.lock export.
  • No source, manifest, Python support, SDK version, workflow, environment or credential changes.
  • Based on current main after aiohttp PR [CD-1788] Patch aiohttp transport vulnerabilities #146 merged through pre-existing repository automation. That rescan closed its three aiohttp findings and exposed native Dependabot alert release: 0.15.0 #43 for certifi.

Validation

Human-controlled rollout

DRAFT intentionally: this repository has an existing workflow that enables auto-merge for admin-authored opened/reopened PRs. Owner decision https://linear.app/casemarkai/issue/CD-1915 tracks a safe manual-remediation opt-out. This agent must not merge, approve, enable auto-merge or request a reviewer to merge. No release publishing or deployment is performed by this work. Require independent current-head review, CI and human merge/release decision; preserve the change in reviewed Stainless seal-back before regeneration. GitHub closure and Vanta ingestion remain separate verification gates.

@thurgood-bot

thurgood-bot Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

DB Cooper Database Review

Risk: very-low

Lockfile-only certifi bump (2026.1.4 to 2026.10.19) in uv.lock and requirements-dev.lock with no source, schema, migration, or query changes; no database risk. Verified repo is a Python API client with no DB layer, schema definition, or migration files.

No database risks found. Migrations and DB-impacting changes look safe.


DB review by DB Cooper | Re-run with /dbreview

@thurgood-bot

thurgood-bot Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Judicial Review

Risk: high

Bumps certifi 2026.1.4 to 2026.10.19 in uv.lock and requirements-dev.lock to remove compromised emSign trust roots (GHSA-6q48-6c7w-9gw2). Scope matches description with no source changes; TLS trust change is security-sensitive and requires human merge/CI verification.

No significant issues found. This code is cleared to ship.


Legal-Grade review by Thurgood | Re-review with /review


Verdict: Human review required · risk high

@thurgood-bot thurgood-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thurgood verdict: Human review required

Risk: high

Bumps certifi 2026.1.4 to 2026.10.19 in uv.lock and requirements-dev.lock to remove compromised emSign trust roots (GHSA-6q48-6c7w-9gw2). Scope matches description with no source changes; TLS trust change is security-sensitive and requires human merge/CI verification.

Lockfile-only certifi bump (2026.1.4 to 2026.10.19) in uv.lock and requirements-dev.lock with no source, schema, migration, or query changes; no database risk. Verified repo is a Python API client with no DB layer, schema definition, or migration files.

  • Draft PRs are not auto-approved

Thurgood is not auto-approving this pull request.

@andrazk andrazk closed this Nov 2, 2026
@andrazk
andrazk deleted the cd-1788-certifi-security branch November 2, 2026 05:33
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