Cleanup some dead code and one instance of null propagation in toolin…

…g NOT userfacing code (#2280)
elastic · Feb 5, 2024 · cd46f5b · cd46f5b
1 parent d598dff
commit cd46f5b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/src/Elastic.Apm.Specification/Validator.cs b/src/Elastic.Apm.Specification/Validator.cs
@@ -284,7 +284,7 @@ private static ImplementationProperty[] GetProperties(Type specType)
 					continue;
 
 				var implementationProperty = new ImplementationProperty(jsonProperty.PropertyName, jsonProperty.PropertyType, specType);
-				var maxLength = (MaxLengthAttribute)(jsonProperty.AttributeProvider.GetAttributes(typeof(MaxLengthAttribute), true).FirstOrDefault());
+				var maxLength = (MaxLengthAttribute)jsonProperty.AttributeProvider?.GetAttributes(typeof(MaxLengthAttribute), true).FirstOrDefault();
 
 				if (maxLength != null)
 					implementationProperty.MaxLength = maxLength.Length;

diff --git a/src/integrations/Elastic.Apm.AspNetCore/Consts.cs b/src/integrations/Elastic.Apm.AspNetCore/Consts.cs
@@ -2,18 +2,13 @@
 // Elasticsearch B.V licenses this file to you under the Apache 2.0 License.
 // See the LICENSE file in the project root for more information
 
-using Microsoft.AspNetCore.Http.Features;
+namespace Elastic.Apm.AspNetCore;
 
-namespace Elastic.Apm.AspNetCore
+internal static class Consts
 {
-	internal static class Consts
+	internal static class OpenIdClaimTypes
 	{
-		internal static class OpenIdClaimTypes
-		{
-			internal const string Email = "email";
-			internal const string UserId = "sub";
-		}
-
-		internal static FormOptions FormContentOptions => new FormOptions { BufferBody = true };
+		internal const string Email = "email";
+		internal const string UserId = "sub";
 	}
 }