feat(develop): Allow to pass user_agent as browser context

[mydea]

Related to #13203

Today, the browser context is inferred in relay from event.request.headers['user-agent']. This is slightly weird in the context of a browser event, as the page is not really the request - we want to replace this with something more explicit.

In order to make this possible, we propose to allow client SDKs to send a partial browser context with only user-agent, and have relay infer the rest of the data from this, if it exists (instead of from event.request).

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
develop-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 2, 2025 2:06pm

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
changelog	⬜️ Ignored (Inspect)			Apr 2, 2025 2:06pm
sentry-docs	⬜️ Ignored (Inspect)			Apr 2, 2025 2:06pm

[lforst]

sgtm. Before we merge this we should actually implement this in Relay.

[mydea]

After talking with @Dav1dde we will actually not do that, because it is hard for relay to partial-update contexts. Instead, relay can use the user-agent from the browser envelope request itself, which should already be here - which is great, because it means the SDK does not need to do/send anything 💪

[lforst]

After talking with @Dav1dde we will actually not do that, because it is hard for relay to partial-update contexts. Instead, relay can use the user-agent from the browser envelope request itself, which should already be here - which is great, because it means the SDK does not need to do/send anything 💪

Scary. In the case of tunneling this information is lost. "Hard for relay to partially update contexts" also seems a bit ridiculous.

[cleptric]

We actually need to figure out how this will be handled in Relay fist.

• If there is no browser context, Relay creates one
• How does the presence of a user_agent fit in in the current behavior
• How do we merge partial context

[lforst]

I think the plan of action for relay is:

• Drop the requiredness of the name field in the Browser context.
• If name and version are not defined and user_agent is defined, backfill name and version in the browser context.
• Keep creating browser context when there is none and the Request interface contains a user agent.

[mydea]

@cleptric are we good with merging this? I think we agreed that this makes sense from a spec-perspective, or should we merge the relay change before we merge this in develop docs?

[cleptric]

I'm good with merging this, if we don't forgot to also add it in Relay 😬