FIX: missing ip settings from SDK package

[lucas-zimerman]

Based on getsentry/sentry-javascript#17364

original context;

This PR fixes a long-standing problem in the SDK where it would set incorrect information about when Relay should (not) infer IP addresses from sent envelope requests.

Previously, this was thought to be controlled by setting event.user.ip_address: '{{auto}}'. However, after an incident in Relay, it was determined that this is in fact not a reliably way to control IP inference. Instead, SDKs should set event.sdk.settings.infer_ip: 'auto' | 'never' (see closes https://github.com/getsentry/sentry-javascript/issues/16252).
Unfortunately, this wasn't implemented immediately but is taken care of in this PR.

(FWIW, the only reason why Relay continued to infer IP addresses for the JS SDK was because it is excempt from logic that would infer IP addresses only if user.ip_address was set to '{{auto}}'. This is necessary to backwards compatibility with older SDKs.)

Follow-ups: We likely also need to adjust the logic in Electron and Lynx (at the very least remove setting user.ip_address).

closes https://github.com/getsentry/sentry-javascript/issues/17351
closes https://github.com/getsentry/sentry-javascript/issues/16252

Instead of bumping o V10, the fix was patched on the current release.

[lucas-zimerman]

This is how the event will look like when you set sendDefaultPii as false

[lucas-zimerman]

auto set https://us.sentry.io/api/0/projects/sentry-sdks/sentry-capacitor/events/dbac1fe89e3b46d6a4b49cd8c1b1593f/json/
never set https://us.sentry.io/api/0/projects/sentry-sdks/sentry-capacitor/events/8f790964c1984a7bab074aa908100bb1/json/

[antonis]

Awesome work @lucas-zimerman 🚀
The changes LGTM. Thank you adding unit test and test data in the comments.

Added a couple of questions but nothing blocking.

[Lms24]

Thanks for fixing this! Looks good to me!

I'm thinking that the SDK might still set user.ip_address: '{{auto}}' in case setDefaultPii: true. In case it does, we could still still clean this up but it's not the end of the world either way. sdk.settings.infer_ip overrules the other field in both cases.

[lucas-zimerman]

Thanks for fixing this! Looks good to me!

I'm thinking that the SDK might still set user.ip_address: '{{auto}}' in case setDefaultPii: true. In case it does, we could still still clean this up but it's not the end of the world either way. sdk.settings.infer_ip overrules the other field in both cases.

I have removed the auto ip here 49d6b67

[lucas-zimerman]

PR test error is unrelated to the added code and will be fixed on another PR

[Lms24]

thanks!