Expected Behavior

Allow more customization or inheritance for CasAuthenticationEntrypoint

Current Behavior

CasAuthenticationEntrypoint has final keyword on

@Override
public final void commence(final HttpServletRequest servletRequest, HttpServletResponse response,
        AuthenticationException authenticationException) throws IOException {
...
}

that prevent developer to change behavior.

If removing final isn't acceptable, otherwise allow changing hardcoded new DefaultRedirectStrategy() inside commence to allow injecting RedirectStrategy inside constructor

Context

Allow developer to apply different strategy without having copying CasAuthenticationEntrypoint, could resolve #3123 for example


I'm totally ok to propose a PR, but I'd like to get validation that is something acceptable

Comment From: rishiraj88

Looks like a nice suggestion. Flexibility should be favoured unless there is a strong blocker.

Comment From: marcusdacoregio

Thanks for the report @kakawait. I've repurposed the ticket to allow customization of the redirect strategy if you are okay with it.