4.29.1 (#9786)

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

* Increase patch 4.29.1

---------

Co-authored-by: Fabio Cavalcante <[email protected]>
Co-authored-by: azfuncgh <[email protected]>

Author: 3 people

build/common.props

@@ -5,7 +5,7 @@
     <LangVersion>latest</LangVersion>
     <MajorVersion>4</MajorVersion>
     <MinorVersion>29</MinorVersion>
-    <PatchVersion>0</PatchVersion>
+    <PatchVersion>1</PatchVersion>
     <BuildNumber Condition="'$(BuildNumber)' == '' ">0</BuildNumber>
     <PreviewVersion></PreviewVersion>
 

src/WebJobs.Script/Extensions/HttpRequestExtensions.cs

@@ -13,7 +13,6 @@
 using Microsoft.AspNetCore.Http.Extensions;
 using Microsoft.AspNetCore.WebUtilities;
 using Microsoft.Azure.WebJobs.Extensions.Http;
-using Microsoft.CodeAnalysis.CSharp.Syntax;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Primitives;
 using Newtonsoft.Json;
@@ -85,9 +84,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)