SDKVersion to hint NETCORE, other minor changes

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 This changelog will be used to generate documentation on [release notes page](http://azure.microsoft.com/documentation/articles/app-insights-release-notes-dotnet/).
 
+## Version 2.10.0
+- SDKVersion modified to be dotnetc for NetCore. This helps identify the source of code path, as implementations are slightly different for NetCore.
+
 ## Version 2.10.0-beta4
 - [Fix NullReferenceException in DiagnosticsEventListener.OnEventWritten](https://github.com/Microsoft/ApplicationInsights-dotnet/issues/1106)
 - [Fix RichPayloadEventSource can get enabled at Verbose level](https://github.com/Microsoft/ApplicationInsights-dotnet/issues/1108)

README.md

@@ -1,4 +1,5 @@
-![Build Status](https://mseng.visualstudio.com/DefaultCollection/_apis/public/build/definitions/96a62c4a-58c2-4dbb-94b6-5979ebc7f2af/1822/badge)
+[![Build Status](https://mseng.visualstudio.com/AppInsights/_apis/build/status/ChuckNorris/AI-BaseSDK-develop-linux?branchName=develop)](https://mseng.visualstudio.com/AppInsights/_build/latest?definitionId=6237?branchName=develop)
+[![Build Status](https://mseng.visualstudio.com/AppInsights/_apis/build/status/ChuckNorris/AI-BaseSDK-GitHub-Master?branchName=develop)](https://mseng.visualstudio.com/AppInsights/_build/latest?definitionId=1822?branchName=develop)
 [![codecov.io](https://codecov.io/github/Microsoft/ApplicationInsights-dotnet/coverage.svg?branch=develop)](https://codecov.io/github/Microsoft/ApplicationInsights-dotnet?branch=develop)
 
 ## NuGet packages

Test/Microsoft.ApplicationInsights.Test/Shared/Metrics/TestUtility/TestUtil.cs

@@ -236,8 +236,11 @@ public static void ValidateSdkVersionString(string versionMoniker)
             Assert.IsNotNull(versionMoniker);
 
             // Expected result example: "m-agg2:2.6.0-12552"
-
+#if NETCOREAPP1_1 || NETCOREAPP2_0
+        const string expectedPrefix = "m-agg2c:";
+#else
             const string expectedPrefix = "m-agg2:";
+#endif        
 
             string sdkRevisionComponentStr = versionMoniker.Substring(expectedPrefix.Length);
             string expectedVersion = SdkVersionUtils.GetSdkVersion(String.Empty);

Test/TestFramework/Shared/EventSourceTest.cs

@@ -93,7 +93,7 @@ private static void VerifyEventMessage(MethodInfo eventMethod, EventWrittenEvent
 
         private static void VerifyEventApplicationName(MethodInfo eventMethod, EventWrittenEventArgs actualEvent)
         {
-#if !NETCOREAPP
+#if !NETCOREAPP1_1
             string expectedApplicationName = AppDomain.CurrentDomain.FriendlyName;
 #else
             string expectedApplicationName = "";

src/Microsoft.ApplicationInsights/Managed/Shared/Extensibility/Implementation/Tracing/HeartbeatProvider.cs

@@ -28,8 +28,12 @@ internal class HeartbeatProvider : IDisposable, IHeartbeatProvider
 
         /// <summary>
         /// Value for property indicating 'app insights version' related specifically to heartbeats.
-        /// </summary>
+        /// </summary>        
+#if NETSTANDARD1_3 || NETSTANDARD2_0
+        private static string sdkVersionPropertyValue = SdkVersionUtils.GetSdkVersion("hbnetc:");
+#else
         private static string sdkVersionPropertyValue = SdkVersionUtils.GetSdkVersion("hbnet:");
+#endif
 
         /// <summary>
         /// The name of the heartbeat metric item and operation context. 

src/Microsoft.ApplicationInsights/Metrics/Implementation/Util.cs

@@ -16,7 +16,11 @@ internal static class Util
 
         private const string FallbackParameterName = "specified parameter";
 
+#if NETSTANDARD1_3 || NETSTANDARD2_0
+        private static string sdkVersionMoniker = SdkVersionUtils.GetSdkVersion("m-agg2c:");
+#else
         private static string sdkVersionMoniker = SdkVersionUtils.GetSdkVersion("m-agg2:");
+#endif
 
         /// <summary>
         /// Parameter check for Null.

src/Microsoft.ApplicationInsights/Microsoft.ApplicationInsights.csproj

@@ -4,7 +4,7 @@
 
   <PropertyGroup>
     <TargetFrameworks>net45;net46;netstandard1.3;netstandard2.0</TargetFrameworks>
-    <TargetFrameworks Condition="$(OS) != 'Windows_NT'">netstandard1.3</TargetFrameworks>
+    <TargetFrameworks Condition="$(OS) != 'Windows_NT'">netstandard1.3;netstandard2.0</TargetFrameworks>
   </PropertyGroup>
 
   <PropertyGroup>

src/Microsoft.ApplicationInsights/TelemetryClient.cs

@@ -22,7 +22,11 @@
     /// </summary>
     public sealed class TelemetryClient
     {
+#if NETSTANDARD1_3 || NETSTANDARD2_0
+        private const string VersionPrefix = "dotnetc:";
+#else
         private const string VersionPrefix = "dotnet:";
+#endif
         private readonly TelemetryConfiguration configuration;
         private string sdkVersion;
 

src/ServerTelemetryChannel/Implementation/TelemetryChannelEventSource.cs

@@ -531,7 +531,7 @@ private static string GetApplicationName()
             string name;
             try
             {
-#if !NETSTANDARD
+#if !NETSTANDARD1_3
                 name = AppDomain.CurrentDomain.FriendlyName;
 #else
                 name = string.Empty;