ValidateNever attribute seems to be ignored

[illunix]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

As you can see in my code I added ValidateNever attribute everywhere I Manufacturer appears

[ValidateNever]
public sealed record GetItemsQuery([property: ValidateNever] Manufacturer? Manufacturer) : IQuery<ItemDto>
{
    public static ValueTask<GetItemsQuery?> BindAsync(
        HttpContext ctx,
        ParameterInfo param
    )
    {
        const string manufacturerKey = "manufacturer";

        Manufacturer? manufacturer = null;
        var manufacturerParam = ctx.Request.Query[manufacturerKey];
        if (!string.IsNullOrEmpty(manufacturerParam))
            manufacturer = Manufacturer.TryFromName(
                manufacturerParam,
                ignoreCase: true,
                out var m
            )
                ? m
                : null;

        return ValueTask.FromResult<GetItemsQuery?>(new(manufacturer));
    }
}

[ValidateNever]
public sealed class Manufacturer : SmartEnum<Manufacturer>
{
    public static readonly Manufacturer Test1 = new("Test 1", 1);
    public static readonly Manufacturer Test2 = new("Test 2", 2);
    public static readonly Manufacturer Test3 = new("Test 3", 3);
    // etc.
}

Expected Behavior

Should skip validating Manufacturer property or whole GetItemsQuery object

Steps To Reproduce

So the package I use for SmartEnum is Ardalis.SmartEnum
Also my Manufacturer class exceedes 32 properties with Manufacturer type

Exceptions (if any)

Maximum validation depth of 32 exceeded at 'Manufacturer.List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0]' in 'Manufacturer'. This is likely caused by a circular reference in the object graph. Consider increasing the MaxDepth in ValidationOptions if deeper validation is required.
System.InvalidOperationException: Maximum validation depth of 32 exceeded at 'Manufacturer.List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0].List[0]' in 'Manufacturer'. This is likely caused by a circular reference in the object graph. Consider increasing the MaxDepth in ValidationOptions if deeper validation is required.

.NET Version

10.0.101

Anything else?

No response

[MihuBot]

I'm a bot. Here are possible related and/or duplicate issues (I may be wrong):

• ValidateNever applied to action parameters is not honored #24241
• ValidateNever applied to action parameters is not honored #44279

[illunix]

So with something like this it seems it skips whole object so that's nice but I need to add this warning disable

#pragma warning disable ASP0029 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.

[SkipValidation]
public sealed record GetItemsQuery