Removing todos and other temp logging

newrelic · Dec 5, 2024 · 275266b · 275266b
1 parent 519c0a2
commit 275266b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/src/Agent/NewRelic/Agent/Extensions/NewRelic.Agent.Extensions/AwsSdk/ArnBuilder.cs b/src/Agent/NewRelic/Agent/Extensions/NewRelic.Agent.Extensions/AwsSdk/ArnBuilder.cs
@@ -3,7 +3,6 @@
 
 using System.Linq;
 using System.Text.RegularExpressions;
-using NewRelic.Agent.Extensions.Collections;
 
 namespace NewRelic.Agent.Extensions.AwsSdk
 {

diff --git a/src/Agent/NewRelic/Agent/Extensions/Providers/Wrapper/AwsSdk/AmazonServiceClientWrapper.cs b/src/Agent/NewRelic/Agent/Extensions/Providers/Wrapper/AwsSdk/AmazonServiceClientWrapper.cs
@@ -39,9 +39,6 @@ public AfterWrappedMethodDelegate BeforeWrappedMethod(InstrumentedMethodCall ins
 
                 // convert the access key to an account id
                 AwsAccountId = AwsAccountIdDecoder.GetAccountId(accessKey);
-
-                // TODO: TESTING ONLY
-                agent.Logger.Info($"Successfully parsed AWS AccountId: {AwsAccountId}");
             }
             catch (Exception e)
             {

diff --git a/src/Agent/NewRelic/Agent/Extensions/Providers/Wrapper/AwsSdk/AwsSdkPipelineWrapper.cs b/src/Agent/NewRelic/Agent/Extensions/Providers/Wrapper/AwsSdk/AwsSdkPipelineWrapper.cs
@@ -49,10 +49,8 @@ private ArnBuilder CreateArnBuilder(IAgent agent, dynamic requestContext)
                     _reportBadArnBuilder = false;
                 }
             }
-            agent.Logger.Debug($"AwsSdkPipelineWrapper: Creating ArnBuilder with partition: {partition}, systemName: {systemName}, accountId: {accountId}");
-            var arnBuilder = new ArnBuilder(partition, systemName, accountId);
-            agent.Logger.Debug($"AwsSdkPipelineWrapper: ArnBuilder created: {arnBuilder}");
-            return arnBuilder;
+
+            return new ArnBuilder(partition, systemName, accountId);
         }
 
         private string GetAccountId(IAgent agent)
@@ -71,8 +69,6 @@ private string GetAccountId(IAgent agent)
                 }
             }
 
-            // TODO: testing only
-            agent.Logger.Debug($"AwsSdkPipelineWrapper: Using accountId: {accountId}");
             return accountId;
         }