From 198607dc857915b5cae318fb0d94fe593a6cb87b Mon Sep 17 00:00:00 2001 From: Matt Raible Date: Sat, 12 Nov 2022 12:36:58 -0700 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Daniel Franco --- .../main/java/package/config/SecurityConfiguration.java.ejs | 1 - .../package/config/SecurityConfiguration_reactive.java.ejs | 6 ------ 2 files changed, 7 deletions(-) diff --git a/generators/server/templates/src/main/java/package/config/SecurityConfiguration.java.ejs b/generators/server/templates/src/main/java/package/config/SecurityConfiguration.java.ejs index 40a2c657e1e9..7ffeb51ce705 100644 --- a/generators/server/templates/src/main/java/package/config/SecurityConfiguration.java.ejs +++ b/generators/server/templates/src/main/java/package/config/SecurityConfiguration.java.ejs @@ -115,7 +115,6 @@ public class SecurityConfiguration { @Value("${spring.security.oauth2.client.provider.oidc.issuer-uri}") private String issuerUri; <%_ } _%> - // private final SecurityProblemSupport problemSupport; public SecurityConfiguration(<% if (authenticationTypeSession && !skipUserManagement) { %>RememberMeServices rememberMeServices, <% } %><% if (!applicationTypeMicroservice) { %>CorsFilter corsFilter, <% } %> JHipsterProperties jHipsterProperties) { <%_ if (authenticationTypeSession && !skipUserManagement) { _%> diff --git a/generators/server/templates/src/main/java/package/config/SecurityConfiguration_reactive.java.ejs b/generators/server/templates/src/main/java/package/config/SecurityConfiguration_reactive.java.ejs index 672c102a5dd9..eaab72d10369 100644 --- a/generators/server/templates/src/main/java/package/config/SecurityConfiguration_reactive.java.ejs +++ b/generators/server/templates/src/main/java/package/config/SecurityConfiguration_reactive.java.ejs @@ -142,7 +142,6 @@ import static org.springframework.security.web.server.util.matcher.ServerWebExch @Configuration @EnableWebFluxSecurity @EnableReactiveMethodSecurity -// @Import(SecurityProblemSupport.class) public class SecurityConfiguration { private final JHipsterProperties jHipsterProperties; @@ -164,7 +163,6 @@ public class SecurityConfiguration { <%_ } _%> <%_ } _%> -// private final SecurityProblemSupport problemSupport; private final CorsWebFilter corsWebFilter; public SecurityConfiguration( @@ -178,7 +176,6 @@ public class SecurityConfiguration { this.clientRegistrationRepository = clientRegistrationRepository; <%_ } _%> this.jHipsterProperties = jHipsterProperties; -// this.problemSupport = problemSupport; this.corsWebFilter = corsWebFilter; } @@ -234,9 +231,6 @@ public class SecurityConfiguration { <%_ if (!skipClient) { _%> .addFilterAfter(new SpaWebFilter(), SecurityWebFiltersOrder.HTTPS_REDIRECT) <%_ } _%> -// .exceptionHandling() -// .accessDeniedHandler(problemSupport) -// .authenticationEntryPoint(problemSupport) <%_ if (authenticationTypeSession) { _%> .formLogin() .requiresAuthenticationMatcher(pathMatchers(HttpMethod.POST, "/api/authentication"))