Skip to content

Limit cumulative PPTX slide content to prevent denial of service - #28

Merged
sudo-py-dev merged 1 commit into
mainfrom
coderabbit/limit-document-text-memory/6a59a7f5
Nov 9, 2026
Merged

sudo-py-dev merged 1 commit into
mainfrom
coderabbit/limit-document-text-memory/6a59a7f5

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Cap cumulative decompressed PPTX slide XML at 128 MiB while retaining the existing per-part limit. Read at most one byte beyond the remaining budget and reject presentations that exceed it.

Add regression coverage for multiple slides, exact-budget acceptance, and exhausted or insufficient budgets.

Validation: Tests were not run.

View coding task

@coderabbitai

coderabbitai Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor Author

Important

Review skipped

This PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ee191909-463f-4621-855e-013d00a5e5c3

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

@sudo-py-dev
sudo-py-dev merged commit 96242f9 into main Nov 9, 2026
8 checks passed
@sudo-py-dev
sudo-py-dev deleted the coderabbit/limit-document-text-memory/6a59a7f5 branch November 9, 2026 11:59
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