Update net45 to net462

.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.

sdk/src/Core/Internal/Context/CallContextContainer.net45.cs → 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())

sdk/src/Core/Internal/Context/HybridContextContainer.net45.cs → 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();

sdk/src/Core/Strategies/DynamicSegmentNamingStrategy.cs

@@ -20,7 +20,7 @@
 using System.Text.RegularExpressions;
 using Amazon.Runtime.Internal.Util;
 using Amazon.XRay.Recorder.Core.Internal.Utils;
-#if NET45
+#if NETFRAMEWORK
 using System.Net.Http;
 #else
 using Microsoft.AspNetCore.Http;
@@ -80,7 +80,7 @@ public DynamicSegmentNamingStrategy(string fallbackSegmentName, string hostNameP
         /// </summary>
         public string FallbackSegmentName { get; set; }
 
-#if NET45
+#if NETFRAMEWORK
         /// <summary>
         /// Gets the name of the segment.
         /// </summary>

sdk/src/Core/Strategies/FixedSegmentNamingStrategy.cs

@@ -18,7 +18,7 @@
 using System;
 
 using Amazon.Runtime.Internal.Util;
-#if NET45
+#if NETFRAMEWORK
 using System.Net.Http;
 #else
 using Microsoft.AspNetCore.Http;
@@ -60,7 +60,7 @@ public FixedSegmentNamingStrategy(string fixedName)
         /// </summary>
         public string FixedName { get; set; }
 
-#if NET45
+#if NETFRAMEWORK
         /// <summary>
         /// Gets the name of the segment.
         /// </summary>

sdk/src/Core/Strategies/SegmentNamingStrategy.cs

@@ -18,7 +18,7 @@
 
 using System;
 
-#if NET45
+#if NETFRAMEWORK
 using System.Net.Http;
 #else
 using Microsoft.AspNetCore.Http;
@@ -45,7 +45,7 @@ public static string GetSegmentNameFromEnvironmentVariable()
             return Environment.GetEnvironmentVariable(SegmentNamingStrategy.EnvironmentVariableSegmentName);
         }
 
-#if NET45
+#if NETFRAMEWORK
         /// <summary>
         /// Gets the name of the segment.
         /// </summary>

sdk/src/Handlers/AspNet/AWSXRayRecorder.Handlers.AspNet.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45</TargetFrameworks>
+    <TargetFrameworks>net462</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>
@@ -33,7 +33,7 @@
     <ProjectReference Include="..\..\Core\AWSXRayRecorder.Core.csproj" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'net45'">
+  <ItemGroup Condition="'$(TargetFramework)' == 'net462'">
     <Reference Include="Microsoft.CSharp" />
   </ItemGroup>
 

sdk/src/Handlers/AwsSdk/AWSXRayRecorder.Handlers.AwsSdk.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netstandard2.0</TargetFrameworks>
+    <TargetFrameworks>net462;netstandard2.0</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>
@@ -37,7 +37,7 @@
     <EmbeddedResource Include="DefaultAWSWhitelist.json" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' != 'net45'">
+  <ItemGroup Condition="'$(TargetFramework)' != 'net462'">
     <Compile Remove="AWSSdkTracingHandler.cs" />
   </ItemGroup>
 

sdk/src/Handlers/EntityFramework/AWSXRayRecorder.Handlers.EntityFramework.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netstandard2.0</TargetFrameworks>
+    <TargetFrameworks>net462;netstandard2.0</TargetFrameworks>
     <Company>Amazon.com, Inc</Company>
     <Product>Amazon Web Service X-Ray Recorder</Product>
     <Copyright>Copyright 2017-2020 Amazon.com, Inc. or its affiliates. All Rights Reserved.</Copyright>
@@ -33,10 +33,10 @@
   <ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
     <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Version="3.1.0" />
     <Compile Remove="AWSXRayEntityFramework6.cs" />
-    <Compile Remove="EFInterceptor.net45.cs" />
+    <Compile Remove="EFInterceptor.netframework.cs" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'net45'">
+  <ItemGroup Condition="'$(TargetFramework)' == 'net462'">
     <PackageReference Include="EntityFramework" Version="6.2.0" />
     <Compile Remove="AWSXRayInterceptorExtensions.cs" />
     <Compile Remove="EFInterceptor.cs" />

sdk/src/Handlers/EntityFramework/EFInterceptor.net45.cs → sdk/src/Handlers/EntityFramework/EFInterceptor.netframework.cs

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

sdk/src/Handlers/EntityFramework/EFUtil.cs

@@ -20,7 +20,7 @@
 using System.Text.RegularExpressions;
 using Amazon.Runtime.Internal.Util;
 using Amazon.XRay.Recorder.Core;
-#if NET45
+#if NETFRAMEWORK
 using Amazon.XRay.Recorder.Core.Internal.Utils;
 #endif
 
@@ -191,7 +191,7 @@ private static string RemovePortNumberFromDataSource(string dataSource)
             return _portNumberRegex.Replace(dataSource, string.Empty);
         }
 
-#if !NET45
+#if !NETFRAMEWORK
         private static bool ShouldCollectSqlText(bool? collectSqlQueriesOverride)
             => collectSqlQueriesOverride ?? _recorder.XRayOptions.CollectSqlQueries;
 #else

sdk/src/Handlers/SqlServer/AWSXRayRecorder.Handlers.SqlServer.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netstandard2.0</TargetFrameworks>
+    <TargetFrameworks>net462;netstandard2.0</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>
@@ -33,8 +33,8 @@
     <ProjectReference Include="..\..\Core\AWSXRayRecorder.Core.csproj" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' != 'net45'">
-    <Compile Remove="TraceableSqlCommand.net45.cs" />
+  <ItemGroup Condition="'$(TargetFramework)' != 'net462'">
+    <Compile Remove="TraceableSqlCommand.netframework.cs" />
   </ItemGroup>
   <ItemGroup Condition="'$(TargetFramework)' != 'netstandard2.0'">
     <Compile Remove="TraceableSqlCommand.netstandard.cs" />

sdk/src/Handlers/SqlServer/DbCommandInterceptor.cs

@@ -150,7 +150,7 @@ protected virtual void CollectSqlInformation(DbCommand command)
         private string BuildSubsegmentName(DbCommand command) 
             => command.Connection.Database + "@" + SqlUtil.RemovePortNumberFromDataSource(command.Connection.DataSource);
 
-#if !NET45
+#if !NETFRAMEWORK
         private bool ShouldCollectSqlText() 
             => _collectSqlQueriesOverride ?? _recorder.XRayOptions.CollectSqlQueries;
 #else

sdk/src/Handlers/SqlServer/TraceableSqlCommand.net45.cs → sdk/src/Handlers/SqlServer/TraceableSqlCommand.netframework.cs

@@ -1,5 +1,5 @@
 //-----------------------------------------------------------------------------
-// <copyright file="TraceableSqlCommand.net45.cs" company="Amazon.com">
+// <copyright file="TraceableSqlCommand.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/Handlers/System.Net/AWSXRayRecorder.Handlers.System.Net.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netstandard2.0</TargetFrameworks>
+    <TargetFrameworks>net462;netstandard2.0</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>

sdk/test/IntegrationTests/AWSXRayRecorder.IntegrationTests.csproj

@@ -2,7 +2,7 @@
 
 
   <PropertyGroup>
-    <TargetFrameworks>net45;netcoreapp2.0;netcoreapp3.1</TargetFrameworks>
+    <TargetFrameworks>net462;netcoreapp2.0;netcoreapp3.1</TargetFrameworks>
     <Company>Amazon.com, Inc</Company>
     <Product>Amazon Web Service X-Ray Recorder</Product>
     <Copyright>Copyright 2016 Amazon.com, Inc. or its affiliates. All Rights Reserved.</Copyright>
@@ -21,9 +21,9 @@
     <PackageReference Include="Moq" Version="4.7.137" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.3.0" />
-    <PackageReference Include="MSTest.TestAdapter" Version="1.1.18" />
-    <PackageReference Include="MSTest.TestFramework" Version="1.1.18" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.1.0" />
+    <PackageReference Include="MSTest.TestAdapter" Version="2.2.8" />
+    <PackageReference Include="MSTest.TestFramework" Version="2.2.8" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\..\src\Core\AWSXRayRecorder.Core.csproj" />

sdk/test/IntegrationTests/TestBase.cs

@@ -44,7 +44,7 @@ protected static void ClassCleanup()
             Recorder.Dispose();
         }
 
-#if NET45
+#if NETFRAMEWORK
         protected BatchGetTracesResponse BatchGetTraces(string traceId)
         {
             var request = new BatchGetTracesRequest();