Record request remote_ip on audit event

The following discussion from !30970 (merged) should be addressed:

  • @stanhu started a discussion: (+2 comments)

    I think this is the last IP used to sign-in by the user. If the user uses an existing session, then this doesn't get updated. I wonder if we also want to log request.remote_ip.

Proposal

  • Ensure ip_address argument precedence over author current_sign_in_ip in AuditEventService
  • [-] Check all current callers of AuditEventService to ensure request.remote_ip is passed into the AuditEventService --> moved to another issue #239305 (closed).
Edited by Tan Le