update net45 to net462

Author: jon-armen

.github/workflows/ci.yml

@@ -20,7 +20,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                version: [net452, netcoreapp2.0, netcoreapp3.1]
+                version: [net462, netcoreapp2.0, netcoreapp3.1]
 
         steps:
         - name: Checkout repository

sdk/Directory.Build.props

@@ -4,7 +4,7 @@
   </ItemGroup>
 
   <!-- NetFramework does not support embedded pdb -->
-  <PropertyGroup Condition="'$(TargetFramework)' == 'net45' or '$(TargetFramework)' == 'net452'">
+  <PropertyGroup Condition="'$(TargetFramework)' == 'net462'">
     <DebugType>portable</DebugType>
   </PropertyGroup>
 </Project>

sdk/src/Core/AWSXRayRecorder.Core.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netstandard2.0;netcoreapp3.1</TargetFrameworks>
+    <TargetFrameworks>net462;netstandard2.0;netcoreapp3.1</TargetFrameworks>
     <Company>Amazon.com, Inc</Company>
     <Product>Amazon Web Service X-Ray Recorder</Product>
     <Copyright>Copyright 2017-2018 Amazon.com, Inc. or its affiliates. All Rights Reserved.</Copyright>
@@ -51,14 +51,14 @@
     <FrameworkReference Include="Microsoft.AspNetCore.App" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' != 'net45'">
-    <Compile Remove="Internal\Context\CallContextContainer.net45.cs" />
-    <Compile Remove="Internal\Context\HybridContextContainer.net45.cs" />
-    <Compile Remove="Internal\Utils\AppSettings.net45.cs" />
+  <ItemGroup Condition="'$(TargetFramework)' != 'net462'">
+    <Compile Remove="Internal\Context\CallContextContainer.netframework.cs" />
+    <Compile Remove="Internal\Context\HybridContextContainer.netframework.cs" />
+    <Compile Remove="Internal\Utils\AppSettings.netframework.cs" />
     <Compile Remove="AWSXRayRecorder.cs" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'net45'">
+  <ItemGroup Condition="'$(TargetFramework)' == 'net462'">
     <Compile Remove="Internal\Context\AsyncLocalContextContainer.netstandard.cs" />
     <Compile Remove="Internal\Context\LambdaContextContainer.netstandard.cs" />
     <Compile Remove="Internal\Utils\AppSettings.netcore.cs" />
@@ -67,7 +67,7 @@
     <Compile Remove="FacadeSegment.cs" />
   </ItemGroup>
 
-  <ItemGroup Condition=" '$(TargetFramework)' == 'net45'">
+  <ItemGroup Condition="'$(TargetFramework)' == 'net462'">
     <Reference Include="System.Configuration" />
     <Reference Include="System.Web" />
     <Reference Include="System.Net.Http" />

sdk/src/Core/AWSXRayRecorderImpl.cs

@@ -37,7 +37,7 @@ namespace Amazon.XRay.Recorder.Core
     public abstract class AWSXRayRecorderImpl : IAWSXRayRecorder
     {
         private static readonly Logger _logger = Logger.GetLogger(typeof(AWSXRayRecorderImpl));
-#if NET45
+#if NETFRAMEWORK
         private static Lazy<AWSXRayRecorder> _lazyDefaultRecorder = new Lazy<AWSXRayRecorder>(() => AWSXRayRecorderBuilder.GetDefaultBuilder().Build());
         protected static Lazy<AWSXRayRecorder> LazyDefaultRecorder
         {
@@ -151,7 +151,7 @@ public ContextMissingStrategy ContextMissingStrategy
         /// <exception cref="ArgumentNullException">The argument has a null value.</exception>
         public void BeginSegment(string name, string traceId = null, string parentId = null, SamplingResponse samplingResponse = null, DateTime? timestamp = null)
         {
-#if !NET45
+#if !NETFRAMEWORK
             if (AWSXRayRecorder.IsLambda())
             {
                 throw new UnsupportedOperationException("Cannot override Facade Segment. New segment not created.");
@@ -193,7 +193,7 @@ public void BeginSegment(string name, string traceId = null, string parentId = n
         /// <exception cref="EntityNotAvailableException">Entity is not available in trace context.</exception>
         public void EndSegment(DateTime? timestamp = null)
         {
-#if !NET45
+#if !NETFRAMEWORK
             if (AWSXRayRecorder.IsLambda())
             {
                 throw new UnsupportedOperationException("Cannot override Facade Segment. New segment not created.");
@@ -727,7 +727,7 @@ protected void PopulateContexts()
             // Prepare XRay section for runtime context
             var xrayContext = new ConcurrentDictionary<string, string>();
 
-#if NET45
+#if NETFRAMEWORK
             xrayContext["sdk"] = "X-Ray for .NET";
 #else
             xrayContext["sdk"] = "X-Ray for .NET Core";
@@ -743,7 +743,7 @@ protected void PopulateContexts()
             }
 
             RuntimeContext["xray"] = xrayContext;
-#if NET45
+#if NETFRAMEWORK
             ServiceContext["runtime"] = ".NET Framework";
 #else
             ServiceContext["runtime"] = ".NET Core Framework";

sdk/src/Core/AwsXrayRecorderBuilder.cs

@@ -56,7 +56,7 @@ public IReadOnlyList<IPlugin> Plugins
             }
         }
 
-#if NET45
+#if NETFRAMEWORK
         /// <summary>
         /// Initializes <see cref="AWSXRayRecorderBuilder"/> instance with default settings.
         /// </summary>
@@ -67,7 +67,7 @@ public static AWSXRayRecorderBuilder GetDefaultBuilder()
         }
 #endif
 
-#if NET45
+#if NETFRAMEWORK
         /// <summary>
         /// Reads plugin settings from app settings, and adds new instance of each plugin into the builder.
         /// If the plugin settings doesn't exist or the value of the settings is invalid, nothing will be added.

…al/Context/CallContextContainer.net45.cs …ext/CallContextContainer.netframework.cssdk/src/Core/Internal/Context/CallContextContainer.net45.cs renamed to sdk/src/Core/Internal/Context/CallContextContainer.netframework.cs sdk/src/Core/Internal/Context/CallContextContainer.net45.cs renamed to sdk/src/Core/Internal/Context/CallContextContainer.netframework.cs

@@ -1,5 +1,5 @@
 //-----------------------------------------------------------------------------
-// <copyright file="CallContextContainer.net45.cs" company="Amazon.com">
+// <copyright file="CallContextContainer.netframework.cs" company="Amazon.com">
 //      Copyright 2016 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 //
 //      Licensed under the Apache License, Version 2.0 (the "License").

sdk/src/Core/Internal/Context/DefaultTraceContext.cs

@@ -25,7 +25,7 @@ public static class DefaultTraceContext
         /// <returns>default instance of <see cref="ITraceContext"/></returns>
         public static ITraceContext GetTraceContext()
         {
-#if NET45
+#if NETFRAMEWORK
             return new CallContextContainer();
 #else
             if (AWSXRayRecorder.IsLambda())

…/Context/HybridContextContainer.net45.cs …t/HybridContextContainer.netframework.cssdk/src/Core/Internal/Context/HybridContextContainer.net45.cs renamed to sdk/src/Core/Internal/Context/HybridContextContainer.netframework.cs sdk/src/Core/Internal/Context/HybridContextContainer.net45.cs renamed to sdk/src/Core/Internal/Context/HybridContextContainer.netframework.cs

@@ -1,5 +1,5 @@
 //-----------------------------------------------------------------------------
-// <copyright file="TraceContext.net45.cs" company="Amazon.com">
+// <copyright file="TraceContext.netframework.cs" company="Amazon.com">
 //      Copyright 2016 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 //
 //      Licensed under the Apache License, Version 2.0 (the "License").

sdk/src/Core/Internal/Emitters/UdpSegmentEmitter.cs

@@ -136,7 +136,7 @@ protected virtual void Dispose(bool disposing)
             {
                 if (_udpClient != null)
                 {
-#if NET45
+#if NETFRAMEWORK
                     _udpClient.Close();
 #else
                     _udpClient.Dispose();