Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-2914

PHPIDS: Allow IPs from the x-forwarded-for header

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.5.0-b1
    • Component/s: None
    • Labels:
      None

      Description

      The PHPIDS plugin references the server's REMOTE_ADDR to determine the client's IP address, but this may not be the case when Blesta is behind a proxy. Instead, this should follow CORE-2349 and retrieve the IP address from Blesta's internal Requestor service

        Issue Links

          Activity

          tyson Tyson Phillips (Inactive) created issue -
          tyson Tyson Phillips (Inactive) made changes -
          Field Original Value New Value
          Link This issue is blocked by CORE-2349 [ CORE-2349 ]
          tyson Tyson Phillips (Inactive) made changes -
          Story Points 1
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.5.0 Sprint 2 [ 67 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          tyson Tyson Phillips (Inactive) made changes -
          Fix Version/s 4.5.0-b1 [ 11108 ]
          Automated transition triggered when Tyson Phillips (Inactive) created a branch in Stash -
          Status Open [ 1 ] In Progress [ 3 ]
          Automated transition triggered when Tyson Phillips (Inactive) created pull request #5 in Stash -
          Status In Progress [ 3 ] In Review [ 5 ]
          Resolution Fixed [ 1 ]
          tyson Tyson Phillips (Inactive) made changes -
          Remaining Estimate 0 minutes [ 0 ]
          Time Spent 41 minutes [ 2460 ]
          Worklog Id 11676 [ 11676 ]
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.5.0 Sprint 2 [ 67 ] 4.5.0 Sprint 2, 4.5.0 Sprint 3 [ 67, 74 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          Automated transition triggered when Tyson Phillips (Inactive) merged pull request #5 in Stash -
          Status In Review [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              tyson Tyson Phillips (Inactive)
              Reporter:
              tyson Tyson Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                31/Jan/19

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 41 minutes
                41m

                  Agile