adjust /authenticate api

jhipster · Aug 23, 2024 · 724442e · 724442e
1 parent ca5ecee
commit 724442e
Show file tree

Hide file tree

Showing 7 changed files with 50 additions and 71 deletions.
diff --git a/generators/app/__snapshots__/generator.spec.ts.snap b/generators/app/__snapshots__/generator.spec.ts.snap
@@ -585,6 +585,7 @@ exports[`generator - app with default config should match snapshot 1`] = `
   ],
   "liquibaseAddH2Properties": false,
   "liquibaseDefaultSchemaName": "",
+  "listOrFlux": "List",
   "lowercaseBaseName": "jhipster",
   "mainClass": "JhipsterApp",
   "mainJavaDir": "src/main/java/",
@@ -682,6 +683,8 @@ exports[`generator - app with default config should match snapshot 1`] = `
   },
   "nodePackageManager": "npm",
   "nodeVersion": "NODE_VERSION",
+  "optionalOrMono": "Optional",
+  "optionalOrMonoOfNullable": "Optional.ofNullable",
   "packageFolder": "com/mycompany/myapp/",
   "packageInfoJavadocs": [
     {
@@ -1197,6 +1200,7 @@ exports[`generator - app with gateway should match snapshot 1`] = `
   ],
   "liquibaseAddH2Properties": false,
   "liquibaseDefaultSchemaName": "",
+  "listOrFlux": "Flux",
   "lowercaseBaseName": "jhipster",
   "mainClass": "JhipsterApp",
   "mainJavaDir": "src/main/java/",
@@ -1294,6 +1298,8 @@ exports[`generator - app with gateway should match snapshot 1`] = `
   },
   "nodePackageManager": "npm",
   "nodeVersion": "NODE_VERSION",
+  "optionalOrMono": "Mono",
+  "optionalOrMonoOfNullable": "Mono.justOrEmpty",
   "packageFolder": "com/mycompany/myapp/",
   "packageInfoJavadocs": [
     {
@@ -1815,6 +1821,7 @@ exports[`generator - app with microservice should match snapshot 1`] = `
   ],
   "liquibaseAddH2Properties": false,
   "liquibaseDefaultSchemaName": "",
+  "listOrFlux": "List",
   "lowercaseBaseName": "jhipster",
   "mainClass": "JhipsterApp",
   "mainJavaDir": "src/main/java/",
@@ -1853,6 +1860,8 @@ exports[`generator - app with microservice should match snapshot 1`] = `
   },
   "nodePackageManager": "npm",
   "nodeVersion": "NODE_VERSION",
+  "optionalOrMono": "Optional",
+  "optionalOrMonoOfNullable": "Optional.ofNullable",
   "packageFolder": "com/mycompany/myapp/",
   "packageInfoJavadocs": [
     {

diff --git a/generators/java/generators/bootstrap/generator.ts b/generators/java/generators/bootstrap/generator.ts
@@ -92,6 +92,13 @@ export default class BootstrapGenerator extends BaseApplicationGenerator {
             ...editFileCallback,
           );
       },
+      imperativeOrReactive({ applicationDefaults }) {
+        applicationDefaults({
+          optionalOrMono: ({ reactive }) => (reactive ? 'Mono' : 'Optional'),
+          optionalOrMonoOfNullable: ({ reactive }) => (reactive ? 'Mono.justOrEmpty' : 'Optional.ofNullable'),
+          listOrFlux: ({ reactive }) => (reactive ? 'Flux' : 'List'),
+        });
+      },
     });
   }
 

diff --git a/generators/server/types.d.ts b/generators/server/types.d.ts
@@ -107,6 +107,9 @@ export type SpringBootApplication = JavaApplication &
     skipCheckLengthOfIdentifier: boolean;
 
     imperativeOrReactive: string;
+    optionalOrMono: string;
+    optionalOrMonoOfNullable: string;
+    listOrFlux: string;
     generateAuthenticationApi?: boolean;
     generateInMemoryUserCredentials?: boolean;
 

diff --git a/generators/spring-boot/templates/src/main/java/_package_/web/rest/AccountResource.java.ejs b/generators/spring-boot/templates/src/main/java/_package_/web/rest/AccountResource.java.ejs
@@ -18,13 +18,13 @@
 -%>
 package <%= packageName %>.web.rest;
 
-  <%_ if (authenticationTypeSession && !reactive) { _%>
+<%_ if (authenticationTypeSession && !reactive) { _%>
 import <%= packageName %>.domain.PersistentToken;
 import <%= packageName %>.repository.PersistentTokenRepository;
-  <%_ } _%>
-  <%_ if (!reactive) { _%>
+<%_ } _%>
+<%_ if (!reactive) { _%>
 import <%= user.entityAbsoluteClass %>;
-  <%_ } _%>
+<%_ } _%>
 import <%= packageName %>.repository.UserRepository;
 import <%= packageName %>.security.SecurityUtils;
 import <%= packageName %>.service.MailService;
@@ -35,31 +35,23 @@ import <%= packageName %>.web.rest.errors.*;
 import <%= packageName %>.web.rest.vm.KeyAndPasswordVM;
 import <%= packageName %>.web.rest.vm.ManagedUserVM;
 
+import java.security.Principal;
 import org.apache.commons.lang3.StringUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpStatus;
 import org.springframework.web.bind.annotation.*;
-  <%_ if (reactive) { _%>
-import org.springframework.web.server.ServerWebExchange;
+<%_ if (reactive) { _%>
+import java.util.Objects;
 import reactor.core.publisher.Mono;
-  <%_ } _%>
-
-  <%_ if (!reactive && !authenticationTypeJwt) { _%>
-import jakarta.servlet.http.HttpServletRequest;
-  <%_ } _%>
+<%_ } else { _%>
+import java.util.*;
+<%_ } _%>
 import jakarta.validation.Valid;
-  <%_ if (authenticationTypeSession && !reactive) { _%>
+<%_ if (authenticationTypeSession && !reactive) { _%>
 import java.nio.charset.StandardCharsets;
 import java.net.URLDecoder;
-  <%_ } _%>
-  <%_ if (reactive) { _%>
-import java.security.Principal;
-import java.util.Objects;
-  <%_ } _%>
-  <%_ if (!reactive) { _%>
-import java.util.*;
-  <%_ } _%>
+<%_ } _%>
 
 /**
  * REST controller for managing the current user's account.
@@ -144,19 +136,13 @@ public class AccountResource {
     /**
      * {@code GET  /authenticate} : check if the user is authenticated, and return its login.
      *
-     * @param request the HTTP request.
+     * @param principal the authentication principal.
      * @return the login if the user is authenticated.
      */
     @GetMapping("/authenticate")
-    <%_ if (reactive) { _%>
-    public Mono<String> isAuthenticated(ServerWebExchange request) {
+    public <%- optionalOrMono %><String> isAuthenticated(Principal principal) {
         LOG.debug("REST request to check if the current user is authenticated");
-        return request.getPrincipal().map(Principal::getName);
-    <%_ } else { _%>
-    public String isAuthenticated(HttpServletRequest request) {
-        LOG.debug("REST request to check if the current user is authenticated");
-        return request.getRemoteUser();
-    <%_ } _%>
+        return <%- optionalOrMonoOfNullable %>(principal).map(Principal::getName);
     }
 
   <%_ } _%>

diff --git a/...rs/spring-boot/templates/src/main/java/_package_/web/rest/AccountResource_oauth2.java.ejs b/...rs/spring-boot/templates/src/main/java/_package_/web/rest/AccountResource_oauth2.java.ejs
@@ -24,21 +24,19 @@ import <%= packageName %>.security.SecurityUtils;
 import <%= packageName %>.service.UserService;
 import <%= packageName %>.service.dto.<%= user.adminUserDto %>;
 
+import java.security.Principal;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.security.authentication.AbstractAuthenticationToken;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 <%_ if (reactive) { _%>
-import org.springframework.web.server.ServerWebExchange;
 import reactor.core.publisher.Mono;
 <%_ } else { _%>
-import jakarta.servlet.http.HttpServletRequest;
+import java.util.Optional;
 <%_ } _%>
 
-import java.security.Principal;
-
 /**
  * REST controller for managing the current user's account.
  */
@@ -82,18 +80,12 @@ public class AccountResource {
     /**
      * {@code GET  /authenticate} : check if the user is authenticated, and return its login.
      *
-     * @param request the HTTP request.
+     * @param principal the authentication principal.
      * @return the login if the user is authenticated.
      */
     @GetMapping("/authenticate")
-<%_ if (reactive) { _%>
-    public Mono<String> isAuthenticated(ServerWebExchange request) {
+    public <%- optionalOrMono %><String> isAuthenticated(Principal principal) {
         LOG.debug("REST request to check if the current user is authenticated");
-        return request.getPrincipal().map(Principal::getName);
-<%_ } else { _%>
-    public String isAuthenticated(HttpServletRequest request) {
-        LOG.debug("REST request to check if the current user is authenticated");
-        return request.getRemoteUser();
-<%_ } _%>
+        return <%- optionalOrMonoOfNullable %>(principal).map(Principal::getName);
     }
 }
diff --git a/...ot/templates/src/main/java/_package_/web/rest/AccountResource_skipUserManagement.java.ejs b/...ot/templates/src/main/java/_package_/web/rest/AccountResource_skipUserManagement.java.ejs
@@ -23,26 +23,21 @@ import <%= packageName %>.security.SecurityUtils;
 import <%= packageName %>.domain.Authority;
 
 <%_ } _%>
+
+import java.security.Principal;
 <%_ if (reactive) { _%>
 import reactor.core.publisher.Mono;
 <%_ } else { _%>
-import jakarta.servlet.http.HttpServletRequest;
+import java.util.Optional;
 <%_ } _%>
-
 <%_ if (authenticationTypeOauth2) { _%>
 import org.springframework.security.oauth2.client.authentication.OAuth2AuthenticationToken;
-  <%_ if (reactive) { _%>
-import org.springframework.web.server.ServerWebExchange;
-  <%_ } _%>
 <%_ } _%>
 <%_ if (authenticationTypeOauth2 || authenticationTypeJwt) { _%>
-import java.security.Principal;
 import org.springframework.security.oauth2.server.resource.authentication.JwtAuthenticationToken;
 import org.springframework.security.authentication.AbstractAuthenticationToken;
 <%_ } else { _%>
   <%_ if (reactive) { _%>
-import java.security.Principal;
-import org.springframework.web.server.ServerWebExchange;
 import org.springframework.security.core.context.ReactiveSecurityContextHolder;
 import org.springframework.security.core.context.SecurityContext;
 import org.springframework.security.core.userdetails.UserDetails;
@@ -147,19 +142,13 @@ public class AccountResource {
     /**
      * {@code GET  /authenticate} : check if the user is authenticated, and return its login.
      *
-     * @param request the HTTP request.
+     * @param principal the authentication principal.
      * @return the login if the user is authenticated.
      */
     @GetMapping("/authenticate")
-  <%_ if (reactive) { _%>
-    public Mono<String> isAuthenticated(ServerWebExchange request) {
+    public <%- optionalOrMono %><String> isAuthenticated(Principal principal) {
         LOG.debug("REST request to check if the current user is authenticated");
-        return request.getPrincipal().map(Principal::getName);
-  <%_ } else { _%>
-    public String isAuthenticated(HttpServletRequest request) {
-        LOG.debug("REST request to check if the current user is authenticated");
-        return request.getRemoteUser();
-  <%_ } _%>
+        return <%- optionalOrMonoOfNullable %>(principal).map(Principal::getName);
     }
 <%_ } _%>
 

diff --git a/...rs/spring-boot/templates/src/main/java/_package_/web/rest/AuthenticateController.java.ejs b/...rs/spring-boot/templates/src/main/java/_package_/web/rest/AuthenticateController.java.ejs
@@ -43,13 +43,12 @@ import org.springframework.security.oauth2.jwt.JwtClaimsSet;
 import org.springframework.security.oauth2.jwt.JwtEncoder;
 import org.springframework.security.oauth2.jwt.JwtEncoderParameters;
 import org.springframework.web.bind.annotation.*;
-<%_ if (reactive) { _%>
 import java.security.Principal;
+<%_ if (reactive) { _%>
 import reactor.core.publisher.Mono;
-import org.springframework.web.server.ServerWebExchange;
 import org.springframework.security.authentication.ReactiveAuthenticationManager;
 <%_ } else { _%>
-import jakarta.servlet.http.HttpServletRequest;
+import java.util.Optional;
 import org.springframework.security.config.annotation.authentication.builders.AuthenticationManagerBuilder;
 import org.springframework.security.core.context.SecurityContextHolder;
 <%_ } _%>
@@ -120,19 +119,13 @@ public class AuthenticateController {
     /**
      * {@code GET /authenticate} : check if the user is authenticated, and return its login.
      *
-     * @param request the HTTP request.
+     * @param principal the authentication principal.
      * @return the login if the user is authenticated.
      */
     @GetMapping("/authenticate")
-<%_ if (reactive) { _%>
-    public Mono<String> isAuthenticated(ServerWebExchange request) {
+    public <%- optionalOrMono %><String> isAuthenticated(Principal principal) {
         LOG.debug("REST request to check if the current user is authenticated");
-        return request.getPrincipal().map(Principal::getName);
-<%_ } else { _%>
-    public String isAuthenticated(HttpServletRequest request) {
-        LOG.debug("REST request to check if the current user is authenticated");
-        return request.getRemoteUser();
-<%_ } _%>
+        return <%- optionalOrMonoOfNullable %>(principal).map(Principal::getName);
     }
 
     public String createToken(Authentication authentication, boolean rememberMe) {