Fixing header lookup for perf/consistency (#9776) (#9778)

fabiocav · web-flow · commit 131e50e3ce73 · 2024-01-09T10:57:08.000-08:00
diff --git a/src/WebJobs.Script/Extensions/HttpRequestExtensions.cs b/src/WebJobs.Script/Extensions/HttpRequestExtensions.cs
@@ -86,9 +86,9 @@ public static bool IsAppServiceInternalRequest(this HttpRequest request, IEnviro
             }
 
             // this header will *always* be present on requests originating externally (i.e. going
-            // through the Anatares front end). For requests originating internally it will NOT be
+            // through the Antares front end). For requests originating internally it will NOT be
             // present.
-            return !request.Headers.Keys.Contains(ScriptConstants.AntaresLogIdHeaderName);
+            return !request.Headers.ContainsKey(ScriptConstants.AntaresLogIdHeaderName);
         }
 
         public static bool IsPlatformInternalRequest(this HttpRequest request, IEnvironment environment = null)

Original file line number	Diff line number	Diff line change
`@@ -86,9 +86,9 @@ public static bool IsAppServiceInternalRequest(this HttpRequest request, IEnviro`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`// this header will always be present on requests originating externally (i.e. going`
`89`		`- // through the Anatares front end). For requests originating internally it will NOT be`
	`89`	`+ // through the Antares front end). For requests originating internally it will NOT be`
`90`	`90`	`// present.`
`91`		`- return !request.Headers.Keys.Contains(ScriptConstants.AntaresLogIdHeaderName);`
	`91`	`+ return !request.Headers.ContainsKey(ScriptConstants.AntaresLogIdHeaderName);`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`public static bool IsPlatformInternalRequest(this HttpRequest request, IEnvironment environment = null)`