Skip to content

Fix display of IP addresses in audit logs

Sean McGivern requested to merge fix-audit-event-ip-address-display into master

What does this MR do?

The ip_address column (introduced in !34065 (merged)) returns an IPAddr, which doesn't specify a JSON representation so we get the Rails default of a hash of its instance variables. We need an explicit to_s to show it in human-readable form.

For #224539 (closed).

I noticed this in !34645 (comment 362885409) but did not follow up 😊

Screenshots

Before:

image

After:

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Merge request reports