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 18:46:21 UTC

[logging-log4j2] 03/03: Only call getMacAddressString() when needed.

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

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

commit 4a4ffaffa946d6bd726cce89ef089ce6d15ac52a
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Mon Jul 26 14:45:05 2021 -0400

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

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 c695fbd..0e04806 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,9 @@ public class DockerLookup extends AbstractLookup {
         Container current = null;
         try {
             final URL url = new URL(baseUri + "/containers/json");
-            final String macAddr = NetUtils.getMacAddressString();
 
             if (url.getProtocol().equals(HTTP)) {
+                final String macAddr = NetUtils.getMacAddressString();
                 final ObjectMapper objectMapper = new ObjectMapper();
                 final List<Container> containerList = objectMapper.readValue(url, new TypeReference<List<Container>>(){});