Skip to content

Add a size limit for stored interactions - #1498

Open
Mzack9999 wants to merge 1 commit into
devfrom
interaction-size-limits
Open

Mzack9999 wants to merge 1 commit into
devfrom
interaction-size-limits

Conversation

@Mzack9999

@Mzack9999 Mzack9999 commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Closes #1497

Proposed changes

  • New -max-body-size server flag (default 1mb). HTTP request bodies and SMTP messages larger than the limit are truncated before they are stored, not rejected.
  • SMTP: at most 50 recipients per message, read and write timeouts, and a message is stored once for each bucket instead of once per recipient.

Proof

New tests TestHTTPBodyIsTruncated, TestDefaultMaxBodySize, TestSMTPDataIsTruncated, TestSMTPMessageStoredOncePerBucket and TestSMTPServerLimits. cmd/interactsh-server and pkg/server tests pass.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. 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: de2705c6-7bfa-4fe5-89e6-ee8e26045833

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.

This branch has not been deployed

No deployments
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