You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2021/07/26 14:59:24 UTC

[logging-log4j2] 02/03: Only call getMacAddressString if needed.

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git

commit 3f9a44ead6bcea26178dfddbd7fb0d4bc8e2babd
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Mon Jul 26 09:39:06 2021 -0400

    Only call getMacAddressString if needed.
---
 .../src/main/java/org/apache/logging/log4j/docker/DockerLookup.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/log4j-docker/src/main/java/org/apache/logging/log4j/docker/DockerLookup.java b/log4j-docker/src/main/java/org/apache/logging/log4j/docker/DockerLookup.java
index d63aa92..ba045bb 100644
--- a/log4j-docker/src/main/java/org/apache/logging/log4j/docker/DockerLookup.java
+++ b/log4j-docker/src/main/java/org/apache/logging/log4j/docker/DockerLookup.java
@@ -59,9 +59,8 @@ public class DockerLookup extends AbstractLookup {
         Container current = null;
         try {
             URL url= new URL(baseUri + "/containers/json");
-            String macAddr = NetUtils.getMacAddressString();
-
             if (url.getProtocol().equals(HTTP)) {
+                String macAddr = NetUtils.getMacAddressString();
                 ObjectMapper objectMapper = new ObjectMapper();
                 List<Container> containerList = objectMapper.readValue(url, new TypeReference<List<Container>>(){});