[Instrumentation.AWSLambda] Update to net6.0

src/OpenTelemetry.Instrumentation.AWSLambda/.publicApi/net6.0/PublicAPI.Shipped.txt

@@ -0,0 +1 @@
+#nullable enable

src/OpenTelemetry.Instrumentation.AWSLambda/AWSLambdaWrapper.cs

@@ -23,10 +23,10 @@ public static class AWSLambdaWrapper
     private static readonly AssemblyName AssemblyName = typeof(AWSLambdaWrapper).Assembly.GetName();
 
     [SuppressMessage("StyleCop.CSharp.OrderingRules", "SA1202:ElementsMustBeOrderedByAccess", Justification = "Initialization order.")]
-    internal static readonly string ActivitySourceName = AssemblyName.Name;
+    internal static readonly string? ActivitySourceName = AssemblyName.Name;
 
-    private static readonly Version Version = AssemblyName.Version;
-    private static readonly ActivitySource AWSLambdaActivitySource = new(ActivitySourceName, Version.ToString());
+    private static readonly Version? Version = AssemblyName.Version;
+    private static readonly ActivitySource AWSLambdaActivitySource = new(ActivitySourceName ?? "OpenTelemetry.Instrumentation.AWSLambda", Version?.ToString());
 
     /// <summary>
     /// Gets or sets a value indicating whether AWS X-Ray propagation should be ignored. Default value is false.

src/OpenTelemetry.Instrumentation.AWSLambda/CHANGELOG.md

@@ -9,6 +9,8 @@
   If null state analysis is enabled in your depending project, you may encounter
   new warnings.
   ([#1295](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1295))
+* BREAKING: Target `net6.0` instead of `netstandard2.0`
+  ([#1545](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1545))
 
 ## 1.2.0-beta.1
 

src/OpenTelemetry.Instrumentation.AWSLambda/Implementation/AWSLambdaResourceDetector.cs

@@ -14,14 +14,26 @@ internal sealed class AWSLambdaResourceDetector : IResourceDetector
     /// <returns>Detected resource.</returns>
     public Resource Detect()
     {
-        var resourceAttributes = new List<KeyValuePair<string, object>>
+        var resourceAttributes = new List<KeyValuePair<string, object>>(4)
         {
             new(AWSLambdaSemanticConventions.AttributeCloudProvider, AWSLambdaUtils.GetCloudProvider()),
-            new(AWSLambdaSemanticConventions.AttributeCloudRegion, AWSLambdaUtils.GetAWSRegion()),
-            new(AWSLambdaSemanticConventions.AttributeFaasName, AWSLambdaUtils.GetFunctionName()),
-            new(AWSLambdaSemanticConventions.AttributeFaasVersion, AWSLambdaUtils.GetFunctionVersion()),
         };
 
+        if (AWSLambdaUtils.GetAWSRegion() is { } region)
+        {
+            resourceAttributes.Add(new(AWSLambdaSemanticConventions.AttributeCloudRegion, region));
+        }
+
+        if (AWSLambdaUtils.GetFunctionName() is { } functionName)
+        {
+            resourceAttributes.Add(new(AWSLambdaSemanticConventions.AttributeFaasName, functionName));
+        }
+
+        if (AWSLambdaUtils.GetFunctionVersion() is { } functionVersion)
+        {
+            resourceAttributes.Add(new(AWSLambdaSemanticConventions.AttributeFaasVersion, functionVersion));
+        }
+
         return new Resource(resourceAttributes);
     }
 }

src/OpenTelemetry.Instrumentation.AWSLambda/Implementation/AWSLambdaUtils.cs

@@ -86,17 +86,17 @@ internal static string GetCloudProvider()
         return CloudProvider;
     }
 
-    internal static string GetAWSRegion()
+    internal static string? GetAWSRegion()
     {
         return Environment.GetEnvironmentVariable(AWSRegion);
     }
 
-    internal static string GetFunctionName(ILambdaContext? context = null)
+    internal static string? GetFunctionName(ILambdaContext? context = null)
     {
         return context?.FunctionName ?? Environment.GetEnvironmentVariable(FunctionName);
     }
 
-    internal static string GetFunctionVersion()
+    internal static string? GetFunctionVersion()
     {
         return Environment.GetEnvironmentVariable(FunctionVersion);
     }

src/OpenTelemetry.Instrumentation.AWSLambda/Implementation/AWSMessagingUtils.cs

@@ -127,8 +127,8 @@ internal static PropagationContext ExtractParentContext(SNSEvent.SNSMessage? mes
 
         var body = sqsMessage.Body;
         if (body != null &&
-            body.TrimStart().StartsWith("{", StringComparison.Ordinal) &&
-            body.Contains(SnsMessageAttributes))
+            body.TrimStart().StartsWith('{') &&
+            body.Contains(SnsMessageAttributes, StringComparison.Ordinal))
         {
             try
             {

src/OpenTelemetry.Instrumentation.AWSLambda/OpenTelemetry.Instrumentation.AWSLambda.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>netstandard2.0</TargetFramework>
+    <TargetFramework>net6.0</TargetFramework>
     <Description>AWS Lambda tracing wrapper for OpenTelemetry .NET</Description>
     <PackageTags>$(PackageTags);AWS Lambda</PackageTags>
     <MinVerTagPrefix>Instrumentation.AWSLambda-</MinVerTagPrefix>