resolve comments

Author: satvu

src/WebJobs.Script/Workers/Http/Configuration/HttpWorkerOptions.cs

@@ -16,12 +16,12 @@ public class HttpWorkerOptions
         public int Port { get; set; }
 
         /// <summary>
-        /// Setting that enables the host to rebuild the initial invocation HTTP Request and send the copy to the worker process.
+        /// Gets or sets a value indicating whether the host will rebuild the initial invocation HTTP Request and send the copy to the worker process.
         /// </summary>
         public bool EnableForwardingHttpRequest { get; set; }
 
         /// <summary>
-        /// Setting that enables YARP to proxy the invocation HTTP request to the worker process.
+        /// Gets or sets a value indicating whether the host will proxy the invocation HTTP request to the worker process.
         /// </summary>
         public bool EnableProxyingHttpRequest { get; set; }
 

src/WebJobs.Script/Workers/Http/Configuration/HttpWorkerOptionsSetup.cs

@@ -46,7 +46,7 @@ public void Configure(HttpWorkerOptions options)
                 ConfigureWorkerDescription(options, customHandlerSection);
                 if (options.Type == CustomHandlerType.None)
                 {
-                    // CustomHandlerType.None is only for maintaining backward compatibilty with httpWorker section.
+                    // CustomHandlerType.None is only for maintaining backward compatability with httpWorker section.
                     _logger.LogWarning($"CustomHandlerType {CustomHandlerType.None} is not supported. Defaulting to {CustomHandlerType.Http}.");
                     options.Type = CustomHandlerType.Http;
                 }

src/WebJobs.Script/Workers/Http/DefaultHttpWorkerService.cs

@@ -27,6 +27,7 @@ public class DefaultHttpWorkerService : IHttpWorkerService
         private readonly IHttpProxyService _httpProxyService;
         private readonly ScriptInvocationResult _successfulInvocationResult;
         private readonly Uri _destinationPrefix;
+        private readonly string _userAgentString;
 
         public DefaultHttpWorkerService(IOptions<HttpWorkerOptions> httpWorkerOptions, ILoggerFactory loggerFactory, IEnvironment environment,
             IOptions<ScriptJobHostOptions> scriptHostOptions, IHttpProxyService httpProxyService)
@@ -60,6 +61,7 @@ internal DefaultHttpWorkerService(HttpClient httpClient, IOptions<HttpWorkerOpti
             };
 
             _destinationPrefix = new UriBuilder(WorkerConstants.HttpScheme, WorkerConstants.HostName, _httpWorkerOptions.Port).Uri;
+            _userAgentString = $"{HttpWorkerConstants.UserAgentHeaderValue}/{ScriptHost.Version}";
         }
 
         private static HttpClient CreateHttpClient(IOptions<HttpWorkerOptions> httpWorkerOptions)
@@ -203,10 +205,10 @@ internal void AddHeaders(HttpRequestMessage httpRequest, string invocationId)
         {
             httpRequest.Headers.Add(HttpWorkerConstants.HostVersionHeaderName, ScriptHost.Version);
             httpRequest.Headers.Add(HttpWorkerConstants.InvocationIdHeaderName, invocationId);
-            httpRequest.Headers.UserAgent.ParseAdd($"{HttpWorkerConstants.UserAgentHeaderValue}/{ScriptHost.Version}");
+            httpRequest.Headers.UserAgent.ParseAdd(_userAgentString);
         }
 
-        internal void AddProxyingHeaders(HttpRequest httpRequest, string invocationId)
+        private void AddProxyingHeaders(HttpRequest httpRequest, string invocationId)
         {
             httpRequest.Headers.TryAdd(HttpWorkerConstants.HostVersionHeaderName, ScriptHost.Version);
 
@@ -215,9 +217,7 @@ internal void AddProxyingHeaders(HttpRequest httpRequest, string invocationId)
                 httpRequest.Headers[HttpWorkerConstants.InvocationIdHeaderName] = invocationId;
             }
 
-            var userAgent = $"{HttpWorkerConstants.UserAgentHeaderValue}/{ScriptHost.Version}";
-            httpRequest.Headers.Remove("User-Agent");
-            httpRequest.Headers.Append("User-Agent", userAgent);
+            httpRequest.Headers.UserAgent = _userAgentString;
         }
 
         internal string GetPathValue(HttpWorkerOptions httpWorkerOptions, string functionName, HttpRequest httpRequest)