From 3060d33ccad9e447af7337975488634a82843a5d Mon Sep 17 00:00:00 2001 From: Paulina Hensman Date: Thu, 24 Sep 2020 10:19:42 +0200 Subject: [PATCH] Remove instance of IP logging in NetworkMonitorAutoDetect MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug: b/152283155 Change-Id: I75ce0f2d7107a2c25f5df73a75b505051163399b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/185183 Reviewed-by: Sami Kalliomäki Reviewed-by: Jonas Oreland Commit-Queue: Paulina Hensman Cr-Commit-Position: refs/heads/master@{#32183} --- sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java b/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java index 5594c18f5b..3d233b3423 100644 --- a/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java +++ b/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java @@ -112,7 +112,10 @@ public class NetworkMonitorAutoDetect extends BroadcastReceiver implements Netwo public void onLinkPropertiesChanged(Network network, LinkProperties linkProperties) { // A link property change may indicate the IP address changes. // so forward the new NetworkInformation to the observer. - Logging.d(TAG, "link properties changed: " + linkProperties.toString()); + // + // linkProperties.toString() has PII that cannot be redacted + // very reliably, so do not include in log. + Logging.d(TAG, "link properties changed"); onNetworkChanged(network); }