Skip to content

feat(transport)!: replace the legacy transport API - #1427

Open
giortzisg wants to merge 1 commit into
ref/telemetry-schedulerfrom
feat/envelope-transport
Open

giortzisg wants to merge 1 commit into
ref/telemetry-schedulerfrom
feat/envelope-transport

Conversation

@giortzisg

@giortzisg giortzisg commented Sep 10, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Replace the legacy transport path with envelope-based delivery, supporting built-in and custom transports consistently.

Issues

Changelog Entry Instructions

To add a custom changelog entry, uncomment the section above. Supports:

  • Single entry: just write text
  • Multiple entries: use bullet points
  • Nested bullets: indent 4+ spaces

For more details: custom changelog entries

Reminders

Stack created with GitHub Stacks CLI • Give Feedback 💬

Changelog Entry

  • Replace the event-based transports with envelope-based ones. Custom transports need to implement the new Transport interface. See the transport section of the migration guide for details.
    • Custom transports need to implement SendEnvelope(ctx, envelope) in place of SendEvent(event), and FlushWithContext(ctx) in place of Flush(timeout). The envelope types live in the new public protocol package.
    • The transport Timeout now applies to every request, including when a custom HTTPClient is set.
    • ClientOptions.DisableTelemetryBuffer is removed. The telemetry buffer is always used.
    • Transport BufferSize now counts envelopes instead of events, and defaults to 1000 instead of 30.
// Before
func (t *customTransport) SendEvent(event *sentry.Event) {}
func (t *customTransport) Flush(timeout time.Duration) bool { return true }

// After
func (t *customTransport) SendEnvelope(ctx context.Context, envelope *protocol.Envelope) error { return nil }
func (t *customTransport) FlushWithContext(ctx context.Context) bool { return true }

@giortzisg
giortzisg added this pull request to stack #1428 September 10, 2026 10:42

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread internal/telemetry/scheduler.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 8bda8ce to f78b14b Compare September 11, 2026 07:19
Comment thread client.go Outdated

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread internal/telemetry/scheduler.go

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread transport.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 6d3e886 to a91cd13 Compare September 22, 2026 09:32
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from a91cd13 to d03300a Compare September 22, 2026 09:56
Comment thread client.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from d03300a to 3e79081 Compare September 23, 2026 12:26

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread _examples/feature-showcase/main.go
Comment thread transport.go Outdated
Comment thread internal/telemetry/scheduler.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 3e79081 to adbf97e Compare September 29, 2026 09:30

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread client.go
Comment thread internal/telemetry/scheduler.go Outdated
Comment thread internal/telemetry/scheduler.go
Comment thread transport.go
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from adbf97e to 8ddd34d Compare September 29, 2026 09:41

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread internal/telemetry/scheduler.go
Comment thread transport.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 8ddd34d to adac5f9 Compare September 29, 2026 13:23

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread internal/telemetry/scheduler.go Outdated
Comment thread internal/telemetry/scheduler.go
Comment thread internal/telemetry/scheduler.go Outdated
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from a544a42 to 66e4827 Compare October 12, 2026 11:23
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 66e4827 to 213fd89 Compare October 12, 2026 12:58
@giortzisg
giortzisg removed this pull request from stack #1448 October 12, 2026 13:04
@giortzisg
giortzisg added this pull request to stack #1452 October 12, 2026 13:04

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread transport.go
Comment thread internal/telemetry/scheduler.go
Comment thread internal/telemetry/scheduler.go
@giortzisg
giortzisg removed this pull request from stack #1452 October 12, 2026 14:07
@giortzisg
giortzisg added this pull request to stack #1455 October 12, 2026 14:08
@giortzisg
giortzisg removed this pull request from stack #1455 October 12, 2026 14:09
@giortzisg
giortzisg added this pull request to stack #1456 October 12, 2026 14:11
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 213fd89 to 95df8d6 Compare October 12, 2026 15:46

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread internal/telemetry/scheduler.go
Comment thread internal/telemetry/scheduler.go
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 95df8d6 to 3a63a2a Compare October 12, 2026 15:57

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3a63a2a. Configure here.

Comment thread transport.go
Comment thread transport.go
Comment thread transport.go
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 3a63a2a to b7a9852 Compare October 13, 2026 10:15
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from b7a9852 to 7bff0b4 Compare October 13, 2026 15:23
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 7bff0b4 to fbeb38d Compare October 13, 2026 16:27
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from fbeb38d to 0bde7c8 Compare October 15, 2026 17:44
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from 0bde7c8 to f08eeb5 Compare October 15, 2026 18:04
@giortzisg
giortzisg force-pushed the feat/envelope-transport branch from f08eeb5 to e53d573 Compare October 16, 2026 10: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.

2 participants