Minor code optimization. (#9960) (#9998)

Author: kshyju

src/WebJobs.Script.Grpc/MessageExtensions/GrpcMessageConversionExtensions.cs

@@ -111,18 +111,19 @@ internal static async Task<TypedData> ToRpcHttp(this HttpRequest request, ILogge
             var http = new RpcHttp()
             {
                 Url = $"{(request.IsHttps ? "https" : "http")}://{request.Host}{request.Path}{request.QueryString}",
-                Method = request.Method.ToString(),
+                Method = request.Method,
                 RawBody = null
             };
             var typedData = new TypedData
             {
                 Http = http
             };
 
+            var shouldUseNullableValueDictionary = ShouldUseNullableValueDictionary(capabilities);
             foreach (var pair in request.Query)
             {
                 var value = pair.Value.ToString();
-                if (ShouldUseNullableValueDictionary(capabilities))
+                if (shouldUseNullableValueDictionary)
                 {
                     http.NullableQuery.Add(pair.Key, new NullableString { Value = value });
                 }
@@ -137,7 +138,7 @@ internal static async Task<TypedData> ToRpcHttp(this HttpRequest request, ILogge
 
             foreach (var pair in request.Headers)
             {
-                if (ShouldUseNullableValueDictionary(capabilities))
+                if (shouldUseNullableValueDictionary)
                 {
                     http.NullableHeaders.Add(pair.Key.ToLowerInvariant(), new NullableString { Value = pair.Value.ToString() });
                 }
@@ -159,7 +160,7 @@ internal static async Task<TypedData> ToRpcHttp(this HttpRequest request, ILogge
                 {
                     if (pair.Value != null)
                     {
-                        if (ShouldUseNullableValueDictionary(capabilities))
+                        if (shouldUseNullableValueDictionary)
                         {
                             http.NullableParams.Add(pair.Key, new NullableString { Value = pair.Value.ToString() });
                         }

src/WebJobs.Script/Extensions/HttpRequestExtensions.cs

@@ -121,7 +121,6 @@ public static async Task<byte[]> GetRequestBodyAsBytesAsync(this HttpRequest req
 
             byte[] bytes;
             using (var ms = new MemoryStream())
-            using (var reader = new StreamReader(ms))
             {
                 await request.Body.CopyToAsync(ms);
                 bytes = ms.ToArray();

src/WebJobs.Script/Host/FunctionMetadataManager.cs

@@ -24,7 +24,6 @@ public class FunctionMetadataManager : IFunctionMetadataManager
         private const string _functionConfigurationErrorMessage = "Unable to determine the primary function script.Make sure atleast one script file is present.Try renaming your entry point script to 'run' or alternatively you can specify the name of the entry point script explicitly by adding a 'scriptFile' property to your function metadata.";
         private const string _metadataProviderName = "Custom";
         private readonly IServiceProvider _serviceProvider;
-        private readonly ILoggerFactory _loggerFactory;
         private IFunctionMetadataProvider _functionMetadataProvider;
         private bool _isHttpWorker;
         private IEnvironment _environment;
@@ -42,12 +41,11 @@ public FunctionMetadataManager(IOptions<ScriptJobHostOptions> scriptOptions, IFu
             _scriptOptions = scriptOptions;
             _serviceProvider = scriptHostManager as IServiceProvider;
             _functionMetadataProvider = functionMetadataProvider;
-            _loggerFactory = loggerFactory;
             _logger = loggerFactory.CreateLogger(LogCategories.Startup);
             _isHttpWorker = httpWorkerOptions?.Value?.Description != null;
             _environment = environment;
 
-            // Every time script host is re-intializing, we also need to re-initialize
+            // Every time script host is re-initializing, we also need to re-initialize
             // services that change with the scope of the script host.
             scriptHostManager.ActiveHostChanged += (s, e) =>
             {
@@ -82,7 +80,7 @@ public bool TryGetFunctionMetadata(string functionName, out FunctionMetadata fun
         /// <param name="forceRefresh">Forces reload from all providers.</param>
         /// <param name="applyAllowList">Apply functions allow list filter.</param>
         /// <param name="includeCustomProviders">Include any metadata provided by IFunctionProvider when loading the metadata</param>
-        /// <returns> An Immmutable array of FunctionMetadata.</returns>
+        /// <returns> An Immutable array of FunctionMetadata.</returns>
         public ImmutableArray<FunctionMetadata> GetFunctionMetadata(bool forceRefresh, bool applyAllowList = true, bool includeCustomProviders = true, IList<RpcWorkerConfig> workerConfigs = null)
         {
             if (forceRefresh || _servicesReset || _functionMetadataArray.IsDefaultOrEmpty)