Check for 'null' arguments in HttpContextServerVariableExtensions.GetServerVariable (#47828)

MackinnonBuck · web-flow · commit d07f377b6c4c · 2023-04-21T16:32:56.000-07:00
diff --git a/src/Http/Http.Extensions/src/HttpContextServerVariableExtensions.cs b/src/Http/Http.Extensions/src/HttpContextServerVariableExtensions.cs
@@ -21,6 +21,9 @@ public static class HttpContextServerVariableExtensions
     /// </returns>
     public static string? GetServerVariable(this HttpContext context, string variableName)
     {
+        ArgumentNullException.ThrowIfNull(context);
+        ArgumentNullException.ThrowIfNull(variableName);
+        
         var feature = context.Features.Get<IServerVariablesFeature>();
 
         if (feature == null)

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,9 @@ public static class HttpContextServerVariableExtensions`
`21`	`21`	`/// </returns>`
`22`	`22`	`public static string? GetServerVariable(this HttpContext context, string variableName)`
`23`	`23`	`{`
	`24`	`+ ArgumentNullException.ThrowIfNull(context);`
	`25`	`+ ArgumentNullException.ThrowIfNull(variableName);`
	`26`	`+`
`24`	`27`	`var feature = context.Features.Get<IServerVariablesFeature>();`
`25`	`28`
`26`	`29`	`if (feature == null)`