spring-security icon indicating copy to clipboard operation
spring-security copied to clipboard

Align DelegatingAuthenticationConverter Constructors

Open kse-music opened this issue 1 year ago • 2 comments

This change is valuable since new ArrayList<>() and List.of have different behaviors, meaning that these two constructors give two different object states without saying so.

This PR aligns the two constructors.

kse-music avatar Oct 18 '24 05:10 kse-music

Also, @kse-music, I updated the PR title and description to reflect some of the API benefits to this change.

jzheaux avatar Oct 23 '24 18:10 jzheaux

Thanks a lot, @jzheaux. Everything is done as you expected.

kse-music avatar Oct 24 '24 02:10 kse-music

Thanks, @kse-music! This is now merged into main.

jzheaux avatar Oct 25 '24 03:10 jzheaux