Skip to content

Refuse Brotli-compressed Parquet metadata imports - #730

Open
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/cybersecurity-updates-review-6cd8
Open

cursor[bot] wants to merge 1 commit into
mainfrom
cursor/cybersecurity-updates-review-6cd8

Conversation

@cursor

@cursor cursor Bot commented Oct 25, 2026

Copy link
Copy Markdown

Local metadata import reads Parquet with pyarrow. Current pyarrow wheels bundle a Brotli decoder with CVE-2026-111497, an out-of-bounds write. No fixed pyarrow wheel is published.

The client reads the Parquet footer and rejects Brotli compression before it decodes column pages. Snappy, zstd, gzip, and uncompressed Parquet still import.

Tests: pytest tests/data_engine/test_brotli_parquet.py (2 passed).

Open in Web View Automation 

pyarrow wheels bundle a Brotli decoder affected by CVE-2026-111497.
Local metadata import reads the Parquet footer and rejects Brotli compression before page decode.

Co-authored-by: guy <guy@dagshub.com>
@coderabbitai

coderabbitai Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e5adfb01-513f-48a1-a748-1bff792c861c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@guysmoilov
guysmoilov marked this pull request as ready for review October 25, 2026 16:14
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