Skip to content

ref(clients): migrate outbound clients to context scope API - #1405

Open
giortzisg wants to merge 1 commit into
scopes/grpc-server-integrationfrom
scopes/outbound-client-integrations
Open

giortzisg wants to merge 1 commit into
scopes/grpc-server-integrationfrom
scopes/outbound-client-integrations

Conversation

@giortzisg

Copy link
Copy Markdown
Contributor

Description

This migrates outbound clients to the new scopes API.

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 CLIGive Feedback 💬

@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 0f95498 to cbcf501 Compare September 1, 2026 09:13
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from cbcf501 to 4668470 Compare September 1, 2026 12:22
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 4668470 to 121f93a Compare September 1, 2026 12:58
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 121f93a to beb802c Compare September 2, 2026 07:59
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from beb802c to 517f58a Compare September 4, 2026 13:12
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 517f58a to 518e067 Compare September 4, 2026 13:26
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 518e067 to 1e93ee8 Compare September 7, 2026 11:40
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 1e93ee8 to 162baa4 Compare September 7, 2026 11:50
Comment thread grpc/client.go Outdated
if hub == nil {
hub = sentry.CurrentHub().Clone()
ctx = sentry.SetHubOnContext(ctx, hub)
func configureClient(ctx context.Context) bool {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

l: I would at least document the meaning of the return value here

Suggested change
func configureClient(ctx context.Context) bool {
// configureClient sets the SDK identifier on the client and returns whether the
// client is enabled.
func configureClient(ctx context.Context) bool {

@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 162baa4 to 6a98fe2 Compare September 16, 2026 09:14
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 6a98fe2 to 037b9a2 Compare September 17, 2026 08:38
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from 037b9a2 to c480559 Compare September 17, 2026 09:09
Resolve HTTP and gRPC clients, active spans, and propagation headers from the request context. Create child spans only when a native parent is active, and preserve caller requests and metadata when adding headers.

Replace stale Sentry baggage even for frozen-empty DSC, preserve third-party members within header limits, and reject baggage belonging to a different selected trace. Derive W3C headers from the same sampling decision as sentry-trace.
@giortzisg
giortzisg removed this pull request from stack #1420 September 18, 2026 11:17
@giortzisg
giortzisg force-pushed the scopes/outbound-client-integrations branch from c480559 to 7284eaf Compare September 18, 2026 11:18
@giortzisg
giortzisg added this pull request to stack #1433 September 18, 2026 11:22
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