Avoid infinite recursion in BeanValidationBeanRegistrationAotProcessor with recursive generics

[scordio]

Prior to this commit, AOT processing for bean validation failed with StackOverflowError for constraints with fields having recursive generic types. With this change, the processing aborts preemptively when a cycle is detected.

• Fixes AOT processing for bean validation causes StackOverflowError #33936

[bjorntj]

I see this is marked as release 6.2.1 but I guess that is a month away. Anyway to get this fix now?

[sbrannen]

I see this is marked as release 6.2.1 but I guess that is a month away. Anyway to get this fix now?

@bjorntj, we will decide when to release 6.2.1 within the team in the coming days.

[sbrannen]

This has been merged into 6.2.x and main in 5e7b3a3 and slightly revised in 051f1da.

Thanks for the PR and for collaborating with me on this at Hack.Commit.Push 2024! 👍

[edeandrea]

Is there a workaround for this fix or do we need to wait for the next release? Removing the spring-boot-starter-validation dependency isn't a viable workaround in my case.

[sbrannen]

Is there a workaround for this fix or do we need to wait for the next release?

It depends...

If you need the AOT processing to occur, no, there is unfortunately no workaround, and you'll have to wait for the 6.2.1 release which is scheduled for next Thursday.

However, if you just want to run your tests via the build or build your application for standard JVM deployment, you could disable the AOT processing when you run the build.

[edeandrea]

Thanks @sbrannen thats what I thought. In my case I'm running the -PnativeTests in my CI workflow (to ensure that the tests run against a native binary). Thats whats failing. The JVM-only tests run fine (as you stated).