Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • None
    • Community Edition
    • Security
    • None
    • PHP (FPM) 7.2.3, Ubuntu Linux 16.04.4

    Description

      Hey There,

      I just wanted to suggest an improvement. I run a local instance on my server and noticed that when the subdomain I have my instance running on through the cloudflare reverse proxy, that the logged IP addresses show the reverse proxy rather than the user's actual address. 

      Could a small tweak be appended to the code to replace the `

      $_SERVER["REMOTE_ADDR"]

       to a null coalesce operator to support this?

      $_SERVER["HTTP_CF_CONNECTING_IP"] ?? $_SERVER["REMOTE_ADDR"]

      Regards,

      Ely.

      Attachments

        Activity

          [OBS-2643] Add Cloudflare IP Address support.
          landy Mike Stupalov made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          landy Mike Stupalov made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
          landy Mike Stupalov made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          landy Mike Stupalov made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          landy Mike Stupalov made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Elycin Ely Haughie created issue -

          People

            landy Mike Stupalov
            Elycin Ely Haughie
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: