Moving WebJobs.Script to .NET 8 (#10744)

Author: fabiocav

release_notes.md

@@ -10,3 +10,4 @@
   - [Added net9 prelaunch app.](https://github.com/Azure/azure-functions-dotnet-worker/pull/2898)
 - Update the `DefaultHttpProxyService` to better handle client disconnect scenarios (#10688)
   - Replaced `InvalidOperationException` with `HttpForwardingException` when there is a ForwarderError
+- Updated `WebJobs.Script` to target .NET 8 (instead of .NET Standard 2.1)

src/WebJobs.Script/Binding/Http/RawScriptResult.cs

@@ -62,7 +62,7 @@ public async Task ExecuteResultAsync(ActionContext context)
                         }
                         else
                         {
-                            response.Headers.Add(header.Key, header.Value.ToString() ?? string.Empty);
+                            response.Headers.Append(header.Key, header.Value.ToString() ?? string.Empty);
                         }
                     }
                 }

src/WebJobs.Script/Binding/Http/ScriptObjectResult.cs

@@ -39,7 +39,7 @@ internal void AddResponseHeaders(HttpContext context)
                     }
                     else
                     {
-                        response.Headers.Add(header.Key, header.Value.ToString() ?? string.Empty);
+                        response.Headers.Append(header.Key, header.Value.ToString() ?? string.Empty);
                     }
                 }
             }

src/WebJobs.Script/Description/Compilation/CompilationServiceException.cs

@@ -11,7 +11,6 @@
 
 namespace Microsoft.Azure.WebJobs.Script.Description
 {
-    [Serializable]
     public sealed class CompilationServiceException : Exception
     {
         public CompilationServiceException()
@@ -25,9 +24,5 @@ public CompilationServiceException(string message) : base(message)
         public CompilationServiceException(string message, Exception innerException) : base(message, innerException)
         {
         }
-
-        private CompilationServiceException(SerializationInfo info, StreamingContext context) : base(info, context)
-        {
-        }
     }
 }

src/WebJobs.Script/Extensions/AssemblyExtensions.cs

@@ -25,7 +25,9 @@ public static string GetCodeBase(this Assembly assembly)
                 return codeBase;
             }
 
+#pragma warning disable SYSLIB0012 // Type or member is obsolete
             return assembly.CodeBase;
+#pragma warning restore SYSLIB0012 // Type or member is obsolete
         }
     }
 }

src/WebJobs.Script/ExternalStartupException.cs

@@ -9,17 +9,12 @@ namespace Microsoft.Azure.WebJobs.Script
     /// An exception that indicates an issue calling into an external Startup class. This will
     /// prevent the host from starting.
     /// </summary>
-    [Serializable]
     public class ExternalStartupException : Exception
     {
         public ExternalStartupException() { }
 
         public ExternalStartupException(string message) : base(message) { }
 
         public ExternalStartupException(string message, Exception inner) : base(message, inner) { }
-
-        protected ExternalStartupException(
-          System.Runtime.Serialization.SerializationInfo info,
-          System.Runtime.Serialization.StreamingContext context) : base(info, context) { }
     }
 }

src/WebJobs.Script/FunctionConfigurationException.cs

@@ -9,17 +9,12 @@ namespace Microsoft.Azure.WebJobs.Script
     /// An exception that indicates an issue with a function. These exceptions will be caught and
     /// logged, but not cause a host to restart.
     /// </summary>
-    [Serializable]
     public class FunctionConfigurationException : Exception
     {
         public FunctionConfigurationException() { }
 
         public FunctionConfigurationException(string message) : base(message) { }
 
         public FunctionConfigurationException(string message, Exception inner) : base(message, inner) { }
-
-        protected FunctionConfigurationException(
-          System.Runtime.Serialization.SerializationInfo info,
-          System.Runtime.Serialization.StreamingContext context) : base(info, context) { }
     }
 }

src/WebJobs.Script/Host/ProxyFunctionProvider.cs

@@ -100,7 +100,7 @@ private ImmutableArray<FunctionMetadata> LoadFunctionMetadata()
             var functionErrors = new Dictionary<string, ICollection<string>>();
             Collection<ProxyFunctionMetadata> proxies = ReadProxyMetadata(_scriptOptions.Value.RootScriptPath, _logger, functionErrors);
 
-            ImmutableArray<FunctionMetadata> metadata;
+            ImmutableArray<FunctionMetadata> metadata = ImmutableArray<FunctionMetadata>.Empty;
             if (proxies != null && proxies.Any())
             {
                 metadata = proxies.ToImmutableArray<FunctionMetadata>();

src/WebJobs.Script/HostConfigurationException.cs

@@ -9,17 +9,12 @@ namespace Microsoft.Azure.WebJobs.Script
     /// An exception that indicates an issue configuring a ScriptHost. This will
     /// prevent the host from starting.
     /// </summary>
-    [Serializable]
     public class HostConfigurationException : Exception
     {
         public HostConfigurationException() { }
 
         public HostConfigurationException(string message) : base(message) { }
 
         public HostConfigurationException(string message, Exception inner) : base(message, inner) { }
-
-        protected HostConfigurationException(
-          System.Runtime.Serialization.SerializationInfo info,
-          System.Runtime.Serialization.StreamingContext context) : base(info, context) { }
     }
 }

src/WebJobs.Script/HostDisposedException.cs

@@ -11,7 +11,6 @@ namespace Microsoft.Azure.WebJobs.Script
     /// <summary>
     /// An exception that indicates that a service was used on a disposed host.
     /// </summary>
-    [Serializable]
     public class HostDisposedException : ObjectDisposedException
     {
         // For this exception, we want a full stack trace to pinpoint the method
@@ -24,31 +23,13 @@ public HostDisposedException(string disposedObjectName, Exception inner)
             _stackTraceString = GetStackTraceString();
         }
 
-        protected HostDisposedException(SerializationInfo info, StreamingContext context)
-            : base(info, context)
-        {
-            if (info == null)
-            {
-                throw new ArgumentNullException(nameof(info));
-            }
-
-            _stackTraceString = info.GetString("FullStackTraceString");
-        }
-
         public override string StackTrace => _stackTraceString;
 
         public string GetStackTraceString()
         {
             return new StackTrace(true).ToString();
         }
 
-        public override void GetObjectData(SerializationInfo info, StreamingContext context)
-        {
-            base.GetObjectData(info, context);
-
-            info.AddValue("FullStackTraceString", _stackTraceString);
-        }
-
         private static string GetDefaultMessage(string disposedObjectName)
         {
             string message = $"The host is disposed and cannot be used. Disposed object: '{disposedObjectName}'";