一啖焗捞丁
一啖焗捞丁
By this submission, we can implement multiple AbstractPreAuthenticatedProcessingFilter and create different authentication when there are more than one external system, then implement specified AuthenticationProvider to handle it, specified by method...
OK! Thank you for your reply @sjohnr > Thanks for the PR, @kcl-co! Just a heads up we are focused on releasing 5.8 and 6.0, and so may be delayed...
> Once we are able to come around to this, I think it's worth considering `AuthenticationConverter` instead. This encourages composition over inheritance and it is what `AuthenticationFilter` and other filters...
> Once we are able to come around to this, I think it's worth considering `AuthenticationConverter` instead. This encourages composition over inheritance and it is what `AuthenticationFilter` and other filters...
@sjohnr > What do you think about this? I'm understanding you, i have read other filter using convert component from spring security source code, and i know "Composition over inheritance"....
@sjohnr OK! I try to minimize the change.
Hi @sjohnr, should I switch branch upstream/5.6.x , create new branch to commit my modified content and new merge request, that i read from CONTRIBUTING.adoc. > Currently, I commit code...
@sjohnr I also think the code is not elegant if i fix through composition. Although we can do it, we forget that the method `getPreAuthenticatedPrincipal` and `getPreAuthenticatedCredentials` is a abstract...
@sjohnr OK!