[Feat][SDK-594] Compress payload and update payload max size to 1mb

[buongarzoni]

Description of the change

Larger payload limit

The maximum payload size has been raised from 512 KB to 1 MB.

Gzip compression

Payloads are now gzip-compressed by default before being sent to Rollbar. This significantly reduces bandwidth — in practice around 10× on typical JSON error payloads.

Compression can be turned off if needed:

rollbar-java / rollbar-android

Rollbar.init(context, "ACCESS_TOKEN", "production",
    config -> config.compressPayload(false));

rollbar-reactive-streams

ConfigBuilder.withAccessToken("ACCESS_TOKEN")
    .compressPayload(false)
    .build();

▎ Note for custom AsyncHttpClient implementations: if you have plugged in your own AsyncHttpClient via ConfigBuilder.httpClient(...), compression is signalled via request.isCompressionRequested(). Your client will still receive the
▎ full JSON body via request.getBody() — it will just be sent uncompressed unless you handle the flag yourself.

1 - Big uncompressed example payload (841kb)

2 - Big uncompressed example payload exceeding limit (1.27mb)

3- Same payload as 2, but compressed (10kb)

Type of change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Maintenance
• New release

Related issues

Shortcut stories and GitHub issues (delete irrelevant)

• Fix [SC-]
• Fix Remove GSON Dependency #1

Checklists

Development

• Lint rules pass locally
• The code changed/added as part of this pull request has been covered with tests
• All tests related to the changed code pass in development

Code review

• This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
• "Ready for review" label attached to the PR and reviewers assigned
• Issue from task tracker has a link to this pull request
• Changes have been reviewed by at least one other engineer

[linear]

SDK-594 Compress payload and update payload limit

[buongarzoni]

@claude review

[buongarzoni]

@claude review

[claude]

All previously raised feedback has been addressed across follow-up commits — LGTM.

Extended reasoning...

Overview

This PR adds opt-out gzip compression of outbound Rollbar payloads (default on) and raises the max payload size from 512KB to 1MB. It touches the sync sender (SyncSender), the reactive-streams AsyncSender plus its bundled HTTP clients (ReactorAsyncHttpClient, ApacheRequestPublisher), the public AsyncHttpRequest interface, and both ConfigBuilder variants. Tests were added for the gzip-on and gzip-off paths in SyncSenderTest.

Security risks

Low. The change is a content-encoding wrapper around the existing HTTPS POST to the Rollbar ingest endpoint — no new auth surface, no new input parsing, no new sinks. The only behavioral concern (silent default-on for third-party AsyncHttpClient implementations) was resolved by switching to a compression-intent pattern: the request body stays a String and isCompressionRequested() is just a hint, so a pre-PR third-party client keeps working (uncompressed) instead of receiving a null body.

Level of scrutiny

Moderate-to-high warranted on the initial revision (default-on behavior change, public extension point, native off-heap resources via Deflater). After the four follow-up commits the PR is in a clean state and matches what I would expect from a careful implementation: try-with-resources around GZIPOutputStream, try/catch + buffer.release() around the pooled Netty ByteBuf, and a raw-stream-first acquire pattern in SyncSender.sendJson so the underlying connection stream is closed if the gzip header write fails.

Other factors

Every prior review point I raised across three reviews has a corresponding remediation commit (44f2889, be1e055, 1f5f214, 9541f0a), all inline comments are marked resolved, and the bug hunting system found nothing new in the current state. Test coverage exists for both compression-on (decompresses captured bytes and asserts equality) and compression-off (asserts Content-Encoding header is never set) paths in SyncSenderTest. The author has been responsive and methodical — comfortable approving without further human review.

[brianr]

@codex please review

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".