mention source code

Sergey Kanzhelev · Sergey Kanzhelev · commit 3203966c1468 · 2018-10-25T01:22:56.000-07:00
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/BaseHeaderParser.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/BaseHeaderParser.cs
@@ -3,6 +3,7 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/BaseHeaderParser.cs
     internal abstract class BaseHeaderParser<T> : HttpHeaderParser<T>
     {
         protected BaseHeaderParser(bool supportsMultipleValues)
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/GenericHeaderParser.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/GenericHeaderParser.cs
@@ -5,6 +5,7 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/GenericHeaderParser.cs
     internal sealed class GenericHeaderParser<T> : BaseHeaderParser<T>
     {
         private GetParsedValueLengthDelegate getParsedValueLength;
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HeaderUtilities.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HeaderUtilities.cs
@@ -5,6 +5,7 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoption of the code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HeaderUtilities.cs
     internal static class HeaderUtilities
     {
         internal static int GetNextNonEmptyOrWhitespaceIndex(
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpHeaderParser.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpHeaderParser.cs
@@ -3,6 +3,7 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HttpHeaderParser.cs
     internal abstract class HttpHeaderParser<T>
     {
         private bool supportsMultipleValues;
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpParseResult.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpParseResult.cs
@@ -3,6 +3,7 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HttpParseResult.cs
     internal enum HttpParseResult
     {
         /// <summary>
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpRuleParser.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/HttpRuleParser.cs
@@ -2,10 +2,10 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System.Diagnostics.Contracts;
-using System.Text;
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HttpRuleParser.cs
     internal static class HttpRuleParser
     {
         internal const char CR = '\r';
diff --git a/src/Microsoft.AspNet.TelemetryCorrelation/Internal/NameValueHeaderValue.cs b/src/Microsoft.AspNet.TelemetryCorrelation/Internal/NameValueHeaderValue.cs
@@ -5,7 +5,9 @@
 
 namespace Microsoft.AspNet.TelemetryCorrelation
 {
-    // According to the RFC, in places where a "parameter" is required, the value is mandatory
+    // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/NameValueHeaderValue.cs
+
+        // According to the RFC, in places where a "parameter" is required, the value is mandatory
     // (e.g. Media-Type, Accept). However, we don't introduce a dedicated type for it. So NameValueHeaderValue supports
     // name-only values in addition to name/value pairs.
     internal class NameValueHeaderValue
diff --git a/test/Microsoft.AspNet.TelemetryCorrelation.Tests/ActivityExtensionsTest.cs b/test/Microsoft.AspNet.TelemetryCorrelation.Tests/ActivityExtensionsTest.cs
@@ -12,7 +12,7 @@ namespace Microsoft.AspNet.TelemetryCorrelation.Tests
     using System.Linq;
     using Xunit;
 
-    public class ActivityExtensionsTest
+    public class NameValueHeaderValueTest
     {
         private const string TestActivityName = "Activity.Test";
 
diff --git a/test/Microsoft.AspNet.TelemetryCorrelation.Tests/Microsoft.AspNet.TelemetryCorrelation.Tests.csproj b/test/Microsoft.AspNet.TelemetryCorrelation.Tests/Microsoft.AspNet.TelemetryCorrelation.Tests.csproj
@@ -43,19 +43,19 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="Microsoft.Web.Xdt" Version="2.1.1" />
-    <PackageReference Include="System.Diagnostics.DiagnosticSource" Version="4.5.0"/>
-    <PackageReference Include="System.Reactive.Core" Version="3.1.1"/>
-    <PackageReference Include="System.Reactive.Interfaces" Version="3.1.1"/>
-    <PackageReference Include="xunit" Version="2.3.1"/>
-    <PackageReference Include="xunit.abstractions" Version="2.0.1"/>
-    <PackageReference Include="xunit.analyzers" Version="0.7.0"/>
-    <PackageReference Include="xunit.assert" Version="2.3.1"/>
-    <PackageReference Include="xunit.core" Version="2.3.1"/>
-    <PackageReference Include="xunit.extensibility.core" Version="2.3.1"/>
-    <PackageReference Include="xunit.extensibility.execution" Version="2.3.1"/>
-    <PackageReference Include="xunit.runner.msbuild" Version="2.2.0"/>
+    <PackageReference Include="System.Diagnostics.DiagnosticSource" Version="4.5.0" />
+    <PackageReference Include="System.Reactive.Core" Version="3.1.1" />
+    <PackageReference Include="System.Reactive.Interfaces" Version="3.1.1" />
+    <PackageReference Include="xunit" Version="2.3.1" />
+    <PackageReference Include="xunit.abstractions" Version="2.0.1" />
+    <PackageReference Include="xunit.analyzers" Version="0.7.0" />
+    <PackageReference Include="xunit.assert" Version="2.3.1" />
+    <PackageReference Include="xunit.core" Version="2.3.1" />
+    <PackageReference Include="xunit.extensibility.core" Version="2.3.1" />
+    <PackageReference Include="xunit.extensibility.execution" Version="2.3.1" />
+    <PackageReference Include="xunit.runner.msbuild" Version="2.2.0" />
 
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.3.1"/>
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.3.1" />
 
     <PackageReference Include="MicroBuild.Core" Version="0.3.0">
       <PrivateAssets>All</PrivateAssets>

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`
`4`	`4`	`namespace Microsoft.AspNet.TelemetryCorrelation`
`5`	`5`	`{`
	`6`	`+ // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/BaseHeaderParser.cs`
`6`	`7`	`internal abstract class BaseHeaderParser<T> : HttpHeaderParser<T>`
`7`	`8`	`{`
`8`	`9`	`protected BaseHeaderParser(bool supportsMultipleValues)`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace Microsoft.AspNet.TelemetryCorrelation`
`7`	`7`	`{`
	`8`	`+ // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/GenericHeaderParser.cs`
`8`	`9`	`internal sealed class GenericHeaderParser<T> : BaseHeaderParser<T>`
`9`	`10`	`{`
`10`	`11`	`private GetParsedValueLengthDelegate getParsedValueLength;`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace Microsoft.AspNet.TelemetryCorrelation`
`7`	`7`	`{`
	`8`	`+ // Adoption of the code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HeaderUtilities.cs`
`8`	`9`	`internal static class HeaderUtilities`
`9`	`10`	`{`
`10`	`11`	`internal static int GetNextNonEmptyOrWhitespaceIndex(`
Original file line number	Diff line number	Diff line change
`@@ -2,10 +2,10 @@`
`2`	`2`	`// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.`
`3`	`3`
`4`	`4`	`using System.Diagnostics.Contracts;`
`5`		`-using System.Text;`
`6`	`5`
`7`	`6`	`namespace Microsoft.AspNet.TelemetryCorrelation`
`8`	`7`	`{`
	`8`	`+ // Adoptation of code from https://github.com/aspnet/HttpAbstractions/blob/07d115400e4f8c7a66ba239f230805f03a14ee3d/src/Microsoft.Net.Http.Headers/HttpRuleParser.cs`
`9`	`9`	`internal static class HttpRuleParser`
`10`	`10`	`{`
`11`	`11`	`internal const char CR = '\r';`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ namespace Microsoft.AspNet.TelemetryCorrelation.Tests`
`12`	`12`	`using System.Linq;`
`13`	`13`	`using Xunit;`
`14`	`14`
`15`		`- public class ActivityExtensionsTest`
	`15`	`+ public class NameValueHeaderValueTest`
`16`	`16`	`{`
`17`	`17`	`private const string TestActivityName = "Activity.Test";`
`18`	`18`