Skip to content

Use settings.infer_ip instead of ip_address: {{auto}} #16252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mydea opened this issue May 12, 2025 · 0 comments
Open

Use settings.infer_ip instead of ip_address: {{auto}} #16252

mydea opened this issue May 12, 2025 · 0 comments

Comments

@mydea
Copy link
Member

mydea commented May 12, 2025

Description

Today, in the browser SDK we set ip_address: {{auto}} to infer the ip address on the server.

There is now a new way to do this, documented here: https://develop.sentry.dev/sdk/data-model/event-payloads/sdk/#attributes

So we should:

  • Stop setting ip_address: {{auto}}
  • And instead set settings.infer_ip: 'auto' in the event metadata (if default PII is enabled)

This should make it easier to do that, as we can control this in a single place in the event processing pipeline instead of in multiple places (which is necessary today, as we also set this for standalone spans etc).

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

No branches or pull requests

1 participant