Client IP section missing after upgrade

On-Premise
1

After upgrade sentry from 8.1 to 21.7 with new sentry，client ip field is gone.

image

I have upgraded sentry 8.1 → 9.1 → 20 ->21

Any idea what is wrong?

Thanks in advance.

2

any idea guys?

3

That’s a pretty big version bump so hard to pinpoint where this got changed. Could this be related to SENTRY_USE_X_FORWARDED_FOR not working · Issue #22955 · getsentry/sentry · GitHub?

4

Hi, thanks for reply. i try to add SENTRY_USE_X_FORWARDED_FOR=True in environment variable but it not work.

5

This is not an environment variable but a configuration setting in sentry.conf.py. That said it defaults to True. The code that checks this header is here: sentry/proxy.py at 6b56637e5503ab6717cbb005cca84d3e066f97d0 · getsentry/sentry · GitHub

What header is your reverse-proxy using for the client IP information?

6

here is my nginx setting.

        location / {
            proxy_pass http://127.0.0.1:9000;
            proxy_set_header   Host                 $http_host;
            proxy_set_header   X-Forwarded-Proto    $scheme;
            proxy_set_header   X-Forwarded-For      $remote_addr;
            proxy_redirect     off;
            keepalive_timeout 0;
            add_header Strict-Transport-Security "max-age=31536000";
        }

I m getting correct IP in CSP events. But other events like python I m not getting client IP field at all.

7

Did you upgrade SDK version as well? Newer SDK versions don’t send IP address by default due to PII concerns.

P.S. Love the annotations in your screenshot. :grin:

9

nope. SDK is not yet upgraded. if i switch back to old sentry instance running on v 8.1, I can still able to see “Client IP” field with proper IP’s.

10

@untitaker any ideas here?

11

Please check your Security & Privacy settings in both project settings and organization settings.

12

Data Scrubbing is off on both project & organization. still no luck :crying_cat_face:

I even try to setup clean sentry without any old data & settings. Is this “client IP” field is remove from new sentry or it suppose to be there?

13

I even try sentry.io but “Client IP” field is missing there too. Is there any other way I can get IP of machine?

We have many backed component running on multiple machine. IP help us to pinpoint on which machine issue is occurred.

14

@zeeg can you put some light on this please? i see you have done some commit for similar issue long time ago.

15

This topic was automatically closed 90 days after the last reply. New replies are no longer allowed.