Skip to content

[wip] plan for supporting client reports#996

Draft
szokeasaurusrex wants to merge 1 commit intomasterfrom
szokeasaurusrex/plan-client-reports
Draft

[wip] plan for supporting client reports#996
szokeasaurusrex wants to merge 1 commit intomasterfrom
szokeasaurusrex/plan-client-reports

Conversation

@szokeasaurusrex
Copy link
Member

@szokeasaurusrex szokeasaurusrex commented Feb 16, 2026

Ref #996
Ref RUST-98

@github-actions
Copy link

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- [wip] plan for supporting client reports ([#996](https://github.com/getsentry/sentry-rust/pull/996))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against 5975043

@linear
Copy link

linear bot commented Feb 16, 2026

Comment on lines +25 to +26
6. Logs buffering spec also requires a hard cap of 1000 queued logs.
- **Scope note for #882:** keep implementation scope narrow to client reports; track the 1000-cap as a separate follow-up issue unless we discover a low-risk one-line fix while implementing.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relay is optimized for up to 100 logs per envelope. SDKs MUST NOT send more than 100 logs per envelope.

To prevent out-of-memory issues in customer applications, SDKs MUST enforce a hard limit of 1000 queued log events. Any logs added beyond this limit are dropped. SDKs MAY use a lower limit but MUST NOT exceed 1000.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad clanker

Will update this!

Copy link
Member

@lcian lcian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good generally! 🚀

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.

2 participants

Comments