Feature request: include IP option

I’m using @sentry/node in a worker thread in an Electron app. Worker threads in electron don’t have the electron module present so I cannot use @sentry/electron there. I cannot use @sentry/browser either because the threads don’t have a window object or a DOM.

Now all events that are sent from the worker thread don’t include the IP address. @HazAT recommended I should use the beforeSend option in the SDK and manually fetch the IP address and attach it.

This seems like potential for improvement in the library. A includeIP option would significantly reduce complexity here.

[kamilogorek]

@KishanBagaria for @sentry/browser it’s doing automatic backfilling. However, I asked around and apparently we do implement automatic IP fetching for other SDKs with a “magic value”.

import * as Sentry from '@sentry/node';

Sentry.init({
  dsn: "__YOUR_DSN__",
  // the rest of config
});

Sentry.setUser({ ip_address: '{{auto}}' }); // it has to be done just once

Sentry.captureException(new Error('wat'));

This should make it work for you 😃

[kamilogorek]

Awesome, cheers!