From 2f5e1f0703129fb07c5d1565031d12022fc74680 Mon Sep 17 00:00:00 2001
From: Trask Stalnaker <trask.stalnaker@gmail.com>
Date: Fri, 18 Feb 2022 08:54:08 -0800
Subject: [PATCH] Fix another reverse lookup (#5393)

* Fix another reverse lookup

* Update tests to verify NET_PEER_NAME is never captured for http server spans

* Revert "Update tests to verify NET_PEER_NAME is never captured for http server spans"

This reverts commit 44c9e910486990039d45eb9f9f4abfb00370d305.
---
 .../tomcat/common/TomcatNetAttributesGetter.java             | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/instrumentation/tomcat/tomcat-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/tomcat/common/TomcatNetAttributesGetter.java b/instrumentation/tomcat/tomcat-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/tomcat/common/TomcatNetAttributesGetter.java
index ab1441a8eb..469cf667b3 100644
--- a/instrumentation/tomcat/tomcat-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/tomcat/common/TomcatNetAttributesGetter.java
+++ b/instrumentation/tomcat/tomcat-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/tomcat/common/TomcatNetAttributesGetter.java
@@ -22,8 +22,9 @@ public class TomcatNetAttributesGetter implements NetServerAttributesGetter<Requ
   @Override
   @Nullable
   public String peerName(Request request) {
-    request.action(ActionCode.REQ_HOST_ATTRIBUTE, request);
-    return request.remoteHost().toString();
+    // not using request.action(ActionCode.REQ_HOST_ATTRIBUTE, request) since that calls
+    // InetAddress.getHostName() which trigger reverse name lookup
+    return null;
   }
 
   @Override