Skip to content

Fix cargo audit by updating h2 to 0.4.16 - #24467

Merged
alamb merged 1 commit into
apache:mainfrom
alamb:fix-h2-audit
Aug 18, 2026
Merged

alamb merged 1 commit into
apache:mainfrom
alamb:fix-h2-audit

Conversation

@alamb

@alamb alamb commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

N/A -- fixes the security_audit CI failure on main: https://github.com/apache/datafusion/actions/runs/32165471695/job/95803921510

Rationale for this change

The security_audit job on main reports RUSTSEC-2026-0258 (h2 unbounded empty DATA frames) for h2 0.4.13. The advisory is fixed in h2 >= 0.4.16.

What changes are included in this PR?

Update h2 from 0.4.13 to 0.4.16 in Cargo.lock, following the same fix as apache/arrow-rs#10740.

Are these changes tested?

cargo audit --ignore RUSTSEC-2026-0194 --ignore RUSTSEC-2026-0195 passes locally with this change.

Are there any user-facing changes?

No

@alamb alamb added the development-process Related to development process of DataFusion label Aug 18, 2026
@alamb
alamb enabled auto-merge August 18, 2026 20:43
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.22%. Comparing base (19f2e85) to head (9c7aa82).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #24467      +/-   ##
==========================================
- Coverage   81.22%   81.22%   -0.01%     
==========================================
  Files        1113     1113              
  Lines      392250   392250              
  Branches   392250   392250              
==========================================
- Hits       318596   318586      -10     
- Misses      54910    54917       +7     
- Partials    18744    18747       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alamb
alamb added this pull request to the merge queue Aug 18, 2026
Merged via the queue into apache:main with commit f6ad781 Aug 18, 2026
38 checks passed
@alamb
alamb deleted the fix-h2-audit branch August 18, 2026 21:18
alamb added a commit that referenced this pull request Nov 13, 2026
…26352, #26550, #26666) (#27397)

## Which issue does this PR close?

- part of #23805
- Backport of the `Cargo.lock` updates from #24467 (`h2`), #25309 and
#26352 (`rustls`), #26253 (`quinn-proto`), #26550 (`rustls-webpki`), and
#26666 (`ctutils`) to `branch-54`
- Also updates `zstd` to 0.13.4 (`main` fixed the same advisories by
moving to `zstd` 0.14 in #25064 / #26495), and ignores RUSTSEC-2026-0435
(`thrift`), which is only fixed by upgrading to parquet 59 (#22744)
- Related to #27387, where the security audit is failing.

Co-authored-by: Claude Fable 5.2 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion v56.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants