[WIP] Use options validation source generator

Directory.Packages.props

@@ -19,6 +19,7 @@
     <PackageVersion Include="Microsoft.Extensions.Logging" Version="$(MicrosoftExtensionsLoggingVersion)" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="$(MicrosoftExtensionsLoggingAbstractionsVersion)" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="$(MicrosoftExtensionsLoggingConsoleVersion)" />
+    <PackageVersion Include="Microsoft.Extensions.Options" Version="$(MicrosoftExtensionsOptionsVersion)" />
     <PackageVersion Include="Microsoft.FileFormats" Version="$(MicrosoftFileFormatsVersion)" />
     <PackageVersion Include="Microsoft.Identity.Web" Version="$(MicrosoftIdentityWebVersion)" />
     <PackageVersion Include="Microsoft.OpenApi.Readers" Version="$(MicrosoftOpenApiReadersVersion)" />

eng/Versions.props

@@ -86,6 +86,7 @@
     <MicrosoftExtensionsLoggingVersion>$(MicrosoftExtensionsLogging80Version)</MicrosoftExtensionsLoggingVersion>
     <MicrosoftExtensionsLoggingAbstractionsVersion>$(MicrosoftExtensionsLoggingAbstractions80Version)</MicrosoftExtensionsLoggingAbstractionsVersion>
     <MicrosoftExtensionsLoggingConsoleVersion>$(MicrosoftExtensionsLoggingConsole80Version)</MicrosoftExtensionsLoggingConsoleVersion>
+    <MicrosoftExtensionsOptionsVersion>$(MicrosoftExtensionsOptions80Version)</MicrosoftExtensionsOptionsVersion>
     <SystemTextJsonVersion>$(SystemTextJson80Version)</SystemTextJsonVersion>
   </PropertyGroup>
   <PropertyGroup Label=".NET 9 Dependent" Condition=" '$(TargetFramework)' == 'net9.0' ">
@@ -95,6 +96,7 @@
     <MicrosoftExtensionsLoggingVersion>$(MicrosoftExtensionsLogging90Version)</MicrosoftExtensionsLoggingVersion>
     <MicrosoftExtensionsLoggingAbstractionsVersion>$(MicrosoftExtensionsLoggingAbstractions90Version)</MicrosoftExtensionsLoggingAbstractionsVersion>
     <MicrosoftExtensionsLoggingConsoleVersion>$(MicrosoftExtensionsLoggingConsole90Version)</MicrosoftExtensionsLoggingConsoleVersion>
+    <MicrosoftExtensionsOptionsVersion>$(MicrosoftExtensionsOptions90Version)</MicrosoftExtensionsOptionsVersion>
     <SystemTextJsonVersion>$(SystemTextJson90Version)</SystemTextJsonVersion>
     <MicrosoftAspNetCoreOpenApiVersion>$(MicrosoftAspNetCoreApp90Version)</MicrosoftAspNetCoreOpenApiVersion>
   </PropertyGroup>
@@ -105,6 +107,7 @@
     <MicrosoftExtensionsLoggingVersion>$(MicrosoftNETCoreApp100Version)</MicrosoftExtensionsLoggingVersion>
     <MicrosoftExtensionsLoggingAbstractionsVersion>$(MicrosoftNETCoreApp100Version)</MicrosoftExtensionsLoggingAbstractionsVersion>
     <MicrosoftExtensionsLoggingConsoleVersion>$(MicrosoftNETCoreApp100Version)</MicrosoftExtensionsLoggingConsoleVersion>
+    <MicrosoftExtensionsOptionsVersion>$(MicrosoftNETCoreApp100Version)</MicrosoftExtensionsOptionsVersion>
     <SystemTextJsonVersion>$(MicrosoftNETCoreApp100Version)</SystemTextJsonVersion>
     <MicrosoftAspNetCoreOpenApiVersion>$(MicrosoftAspNetCoreApp100Version)</MicrosoftAspNetCoreOpenApiVersion>
   </PropertyGroup>

eng/dependabot/net8.0/Packages.props

@@ -7,6 +7,7 @@
     <PackageReference Include="Microsoft.Extensions.Logging" Version="$(MicrosoftExtensionsLogging80Version)" />
     <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="$(MicrosoftExtensionsLoggingAbstractions80Version)" />
     <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="$(MicrosoftExtensionsLoggingConsole80Version)" />
+    <PackageReference Include="Microsoft.Extensions.Options" Version="$(MicrosoftExtensionsOptions80Version)" />
     <!--
       We want to update "Microsoft.NETCore.App.Runtime.*" but those packages are considered platform packages and cannot be added
       as a package reference. Use Microsoft.NETCore.DotNetHost instead which is released in lockstep with the runtime packages and

eng/dependabot/net8.0/Versions.props

@@ -9,6 +9,8 @@
     <MicrosoftExtensionsLoggingAbstractions80Version>8.0.3</MicrosoftExtensionsLoggingAbstractions80Version>
     <!-- Microsoft.Extensions.Logging.Console -->
     <MicrosoftExtensionsLoggingConsole80Version>8.0.1</MicrosoftExtensionsLoggingConsole80Version>
+    <!-- Microsoft.Extensions.Options -->
+    <MicrosoftExtensionsOptions80Version>8.0.2</MicrosoftExtensionsOptions80Version>
     <!-- Microsoft.NETCore.App -->
     <MicrosoftNETCoreApp80Version>8.0.15</MicrosoftNETCoreApp80Version>
     <!-- System.Text.Json -->

eng/dependabot/net9.0/Versions.props

@@ -9,6 +9,8 @@
     <MicrosoftExtensionsLoggingAbstractions90Version>9.0.4</MicrosoftExtensionsLoggingAbstractions90Version>
     <!-- Microsoft.Extensions.Logging.Console -->
     <MicrosoftExtensionsLoggingConsole90Version>9.0.4</MicrosoftExtensionsLoggingConsole90Version>
+    <!-- Microsoft.Extensions.Options -->
+    <MicrosoftExtensionsOptions90Version>9.0.4</MicrosoftExtensionsOptions90Version>
     <!-- Microsoft.NETCore.App -->
     <MicrosoftNETCoreApp90Version>9.0.4</MicrosoftNETCoreApp90Version>
     <!-- System.Text.Json -->

src/Directory.Build.targets

@@ -126,4 +126,14 @@
     </ItemGroup>
   </Target>
 
+  <!-- Disable options validation source generator, to let us use a checked-in copy
+       of generated sources that include a fix for https://github.com/dotnet/runtime/issues/115347 -->
+  <Target Name="DisableOptionsValidationGenerator"
+          BeforeTargets="CoreCompile"
+          Condition="'$(DisableOptionsValidationGenerator)' == 'true'">
+    <ItemGroup>
+      <Analyzer Remove="@(Analyzer)" Condition="'%(Filename)' == 'Microsoft.Extensions.Options.SourceGeneration'" />
+    </ItemGroup>
+  </Target>
+
 </Project>

src/Microsoft.Diagnostics.Monitoring.Extension.Common/EgressHelper.cs

@@ -190,7 +190,6 @@ private static ServiceCollection CreateServices<TOptions>(ExtensionEgressPayload
                         .Build()
                         .Bind(options);
                 });
-            services.AddSingleton<IValidateOptions<TOptions>, DataAnnotationValidateOptions<TOptions>>();
 
             services.AddSingleton(new EgressProperties(payload.Properties));
 

src/Microsoft.Diagnostics.Monitoring.Extension.Common/Microsoft.Diagnostics.Monitoring.Extension.Common.csproj

@@ -9,10 +9,6 @@
     <IsShippingAssembly>true</IsShippingAssembly>
   </PropertyGroup>
 
-  <ItemGroup>
-    <Compile Include="..\Tools\dotnet-monitor\DataAnnotationValidateOptions.cs" Link="DataAnnotationValidateOptions.cs" />
-  </ItemGroup>
-
   <ItemGroup>
     <InternalsVisibleTo Include="Microsoft.Diagnostics.Monitoring.AzureBlobStorageTests.UnitTests" />
     <InternalsVisibleTo Include="Microsoft.Diagnostics.Monitoring.S3StorageTests.UnitTests" />

src/Microsoft.Diagnostics.Monitoring.Options/FileSystemEgressProviderOptionsValidator.cs

@@ -0,0 +1,19 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System;
+using Microsoft.Extensions.Options;
+
+namespace Microsoft.Diagnostics.Tools.Monitor.Egress.FileSystem
+{
+    [OptionsValidator]
+    internal sealed partial class FileSystemEgressProviderOptionsValidator : IValidateOptions<FileSystemEgressProviderOptions>
+    {
+        IServiceProvider _serviceProvider;
+
+        public FileSystemEgressProviderOptionsValidator(IServiceProvider serviceProvider)
+        {
+            _serviceProvider = serviceProvider;
+        }
+    }
+}

src/Microsoft.Diagnostics.Monitoring.Options/GlobalCounterOptions.cs

@@ -7,6 +7,8 @@
 using System.ComponentModel.DataAnnotations;
 using System.Globalization;
 using System.Linq;
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Options;
 
 namespace Microsoft.Diagnostics.Monitoring.WebApi
 {
@@ -48,23 +50,28 @@ public class GlobalProviderOptions
         public float? IntervalSeconds { get; set; }
     }
 
+    [OptionsValidator]
+    partial class GlobalProviderOptionsValidator : IValidateOptions<GlobalProviderOptions>
+    {
+    }
+
     partial class GlobalCounterOptions : IValidatableObject
     {
         public IEnumerable<ValidationResult> Validate(ValidationContext validationContext)
         {
+            var providerValidator = validationContext.GetRequiredService<IValidateOptions<GlobalProviderOptions>>();
             var results = new List<ValidationResult>();
 
             if (Providers != null)
             {
-                var providerResults = new List<ValidationResult>();
                 foreach ((string provider, GlobalProviderOptions options) in Providers)
                 {
-                    providerResults.Clear();
-                    if (!Validator.TryValidateObject(options, new ValidationContext(options), providerResults, true))
+                    ValidateOptionsResult providerResults = providerValidator.Validate(provider, options);
+                    if (providerResults.Failed)
                     {
                         // We prefix the validation error with the provider.
-                        results.AddRange(providerResults.Select(r => new ValidationResult(
-                            string.Format(CultureInfo.CurrentCulture, OptionsDisplayStrings.ErrorMessage_NestedProviderValidationError, provider, r.ErrorMessage))));
+                        results.AddRange(providerResults.Failures.Select(r => new ValidationResult(
+                            string.Format(CultureInfo.CurrentCulture, OptionsDisplayStrings.ErrorMessage_NestedProviderValidationError, provider, r))));
                     }
                 }
             }
@@ -73,6 +80,17 @@ public IEnumerable<ValidationResult> Validate(ValidationContext validationContex
         }
     }
 
+    [OptionsValidator]
+    partial class GlobalCounterOptionsValidator : IValidateOptions<GlobalCounterOptions>
+    {
+        private readonly IServiceProvider _serviceProvider;
+
+        public GlobalCounterOptionsValidator(IServiceProvider serviceProvider)
+        {
+            _serviceProvider = serviceProvider;
+        }
+    }
+
     internal static class GlobalCounterOptionsExtensions
     {
         public static float GetIntervalSeconds(this GlobalCounterOptions options) =>

src/Microsoft.Diagnostics.Monitoring.Options/MetricsOptions.cs

@@ -1,6 +1,7 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+using Microsoft.Extensions.Options;
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
@@ -43,6 +44,7 @@ public class MetricsOptions
         [Display(
             ResourceType = typeof(OptionsDisplayStrings),
             Description = nameof(OptionsDisplayStrings.DisplayAttributeDescription_MetricsOptions_Providers))]
+        [ValidateEnumeratedItems]
         public List<MetricProvider> Providers { get; set; } = [];
 
         [Display(

src/Microsoft.Diagnostics.Monitoring.Options/MetricsOptionsValidator.cs

@@ -0,0 +1,12 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using Microsoft.Extensions.Options;
+
+namespace Microsoft.Diagnostics.Monitoring.WebApi
+{
+    [OptionsValidator]
+    public partial class MetricsOptionsValidator : IValidateOptions<MetricsOptions>
+    {
+    }
+}

src/Microsoft.Diagnostics.Monitoring.Options/Microsoft.Diagnostics.Monitoring.Options.csproj

@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk.Web">
 
   <PropertyGroup>
     <!--
@@ -9,6 +9,7 @@
     <IsShippingAssembly>true</IsShippingAssembly>
     <OutputType>Library</OutputType>
     <Nullable>enable</Nullable>
+    <DisableOptionsValidationGenerator>true</DisableOptionsValidationGenerator>
   </PropertyGroup>
 
   <ItemGroup>