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

Order client creation does not include IP address in client setting log

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.3.0-b2
    • Component/s: None
    • Labels:
      None

      Description

      When creating a client, settings are also set. A log record is created (CORE-2692) to store what settings were saved and who performed that action, however, the user ID and IP address are not set. The user is in the process of being created so it may not yet be available if the client is being created by themselves through an order form or API. However, the IP address should be set.

      1. Update the Requestor to include the IP address even if the user is not logged in

      When a client is added through the order plugin, a record is also added to the log_client_settings. This log is missing the user_id and ip_address fields are missing data even though it should be available.

        Issue Links

          Activity

          Automated transition triggered when Tyson Phillips (Inactive) merged pull request #469 in Stash -
          Status In Review [ 5 ] Closed [ 6 ]
          tyson Tyson Phillips (Inactive) made changes -
          Time Spent 36 minutes [ 2160 ] 57 minutes [ 3420 ]
          Worklog Id 11274 [ 11274 ]
          tyson Tyson Phillips (Inactive) made changes -
          Link This issue relates to CORE-2692 [ CORE-2692 ]
          tyson Tyson Phillips (Inactive) made changes -
          Description When a client is added through the order plugin, a record is also added to the log_client_settings. This log is missing the user_id and ip_address fields are missing data even though it should be available. When creating a client, settings are also set. A log record is created (CORE-2692) to store what settings were saved and who performed that action, however, the user ID and IP address are not set. The user is in the process of being created so it may not yet be available if the client is being created by themselves through an order form or API. However, the IP address should be set.

          # Update the Requestor to include the IP address even if the user is not logged in

          ----

          When a client is added through the order plugin, a record is also added to the log_client_settings. This log is missing the user_id and ip_address fields are missing data even though it should be available.
          tyson Tyson Phillips (Inactive) made changes -
          Summary Order: Client creation makes incomplete setting log Order client creation does not include IP address in client setting log
          Automated transition triggered when Jonathan Reissmueller created pull request #50 in Stash -
          Status In Progress [ 3 ] In Review [ 5 ]
          Resolution Fixed [ 1 ]
          jonathan Jonathan Reissmueller made changes -
          Time Spent 21 minutes [ 1260 ] 36 minutes [ 2160 ]
          Worklog Id 11268 [ 11268 ]
          jonathan Jonathan Reissmueller made changes -
          Remaining Estimate 0 minutes [ 0 ]
          Time Spent 21 minutes [ 1260 ]
          Worklog Id 11268 [ 11268 ]
          Automated transition triggered when Jonathan Reissmueller created a branch in Stash -
          Status Open [ 1 ] In Progress [ 3 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.3.0 Sprint 11 [ 62 ]
          jonathan Jonathan Reissmueller made changes -
          Field Original Value New Value
          Rank Ranked higher
          jonathan Jonathan Reissmueller created issue -

            People

            • Assignee:
              jonathan Jonathan Reissmueller
              Reporter:
              jonathan Jonathan Reissmueller
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                29/Jun/18

                Time Tracking

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

                  Agile