Skip to content

Enforce size limits in feedback envelope processors #79568

@aliu39

Description

@aliu39

The long-term solution / Follow-up for:

Some context: #79326 (comment)

  • Check for message size limits on the envelope schemas, in rust. Add them if they don't exist. Make sure they're <= 4096 chars as specified in the PR above. (For envelopes we prefer to use bytes.)

    • new feedback (user_report_v2)
    • legacy feedback (user_report)
  • Rather than truncate, emit a rejected outcome for Data Category UserReportV2, and return an error for the SDK.

  • Clean up the feedback max-size option and truncate logic in the previous PR.

A separate size limit could be enforced for spam detection, for COGS. But I wouldn't about it for this issue.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions