Updated Controllers

Author: andresWeitzel

src/main/java/com/api/rest/v1/security/controllers/AuthController.java

@@ -79,44 +79,46 @@ public class AuthController {
 	@PostMapping("/signin")
 	public ResponseEntity<?> signin(@Valid @RequestBody SigninUsuarioDTO signinUsuario, BindingResult bindingResult) {
 
-		if (bindingResult.hasErrors()) {
-			return new ResponseEntity<String>("Campos o Email Inválidos", HttpStatus.BAD_REQUEST);
+		if (signinUsuario.getNombre().isBlank() 
+				|| signinUsuario.getApellido().isBlank() 
+				|| signinUsuario.getUsername().isBlank()
+				|| signinUsuario.getPassword().isBlank()
+				|| signinUsuario.getEmail().isBlank()) {
+			return new ResponseEntity<String>("No se permiten campos vacios!!", HttpStatus.BAD_REQUEST);
 		}
 
 		if (usuarioServiceImpl.existsByUsername(signinUsuario.getUsername())) {
-			return new ResponseEntity<String>("El Username del Usuario ya existe en la DB", HttpStatus.BAD_REQUEST);
+			return new ResponseEntity<String>("El Username ya existe en la DB!!", HttpStatus.BAD_REQUEST);
 		}
 
 		if (usuarioServiceImpl.existsByEmail(signinUsuario.getEmail())) {
-			return new ResponseEntity<String>("El Email del Usuario ya existe en la DB", HttpStatus.BAD_REQUEST);
+			return new ResponseEntity<String>("El Email ya existe en la DB!!", HttpStatus.BAD_REQUEST);
 		}
 
-		if (signinUsuario.getNombre().isBlank() || signinUsuario.getApellido().isBlank() || signinUsuario.getUsername().isBlank()
-				|| signinUsuario.getPassword().isBlank() || signinUsuario.getPassword().isBlank()
-				|| signinUsuario.getEmail().isBlank()) {
-			return new ResponseEntity<String>("No se permiten campos vacios", HttpStatus.BAD_REQUEST);
+	
+		if (bindingResult.hasErrors()) {
+			return new ResponseEntity<String>("Campos o Email Inválidos!!", HttpStatus.BAD_REQUEST);
 		}
 
-		Usuario usuario = new Usuario(signinUsuario.getNombre(),signinUsuario.getApellido(), signinUsuario.getUsername(),
-				passwordEncoder.encode(signinUsuario.getPassword()), signinUsuario.getEmail());
+		Usuario usuario = new Usuario(signinUsuario.getNombre(),signinUsuario.getApellido()
+				, signinUsuario.getUsername(), passwordEncoder.encode(signinUsuario.getPassword())
+				, signinUsuario.getEmail());
 
 		Set<TipoRol> roles = new HashSet<>();
 
-		if (signinUsuario.getRoles().contains("user") || signinUsuario.getRoles().contains("admin")
-				|| signinUsuario.getRoles().contains("")) {
-
-			roles.add(TipoRol.ROLE_USER);
-		}
-
-		if (signinUsuario.getRoles().contains("admin")) {
+	
+		if (signinUsuario.getRoles().contains("admin") || signinUsuario.getRoles().contains("ROLE_ADMIN")) {
 			roles.add(TipoRol.ROLE_ADMIN);
+			roles.add(TipoRol.ROLE_USER);
+		}else {
+			roles.add(TipoRol.ROLE_USER);
 		}
 
 		usuario.setRoles(roles);
 
 		usuarioServiceImpl.addUsuario(usuario);
 
-		return new ResponseEntity<String>("Usuario Insertado Correctamente", HttpStatus.CREATED);
+		return new ResponseEntity<SigninUsuarioDTO>(signinUsuario, HttpStatus.CREATED);
 	}
 
 
@@ -147,11 +149,11 @@ public ResponseEntity<?> signin(@Valid @RequestBody SigninUsuarioDTO signinUsuar
 	public ResponseEntity<?> login(@Valid @RequestBody LoginUsuarioDTO loginUsuario, BindingResult bindingResult) {
 
 		if (bindingResult.hasErrors()) {
-			return new ResponseEntity<String>("Campos Inválidos", HttpStatus.BAD_REQUEST);
+			return new ResponseEntity<String>("Campos Inválidos.!!", HttpStatus.BAD_REQUEST);
 		}
 
 		if (!(usuarioServiceImpl.existsByUsername(loginUsuario.getUsername()))) {
-			return new ResponseEntity<String>("El Usuario no existe. Comprobar username y password!!",
+			return new ResponseEntity<String>("Usuario Inexistente. Verificar campos!!",
 					HttpStatus.BAD_REQUEST);
 		}
 

src/main/java/com/api/rest/v1/security/jwt/JwtProvider.java

@@ -38,7 +38,8 @@ public class JwtProvider {
 	    private String secret="secret";
 
 	    //private int expiration= 300000 * 1;//5min * x --> 5min
-	    private int expiration= 300000 * 10;
+	    //private int expiration= 300000 * 10;
+	    private int expiration= 300000 / 5;
 
 
 	    public String generateToken(Authentication authentication){

src/main/java/com/api/rest/v1/security/repositories/I_UsuarioRepository.java

@@ -2,9 +2,9 @@
 
 import java.util.Optional;
 
-import org.bson.types.ObjectId;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
+import org.springframework.data.mongodb.repository.ExistsQuery;
 import org.springframework.data.mongodb.repository.MongoRepository;
 import org.springframework.data.mongodb.repository.Query;
 import org.springframework.stereotype.Repository;
@@ -31,6 +31,7 @@ public interface I_UsuarioRepository extends MongoRepository<Usuario, String> {
 
 	@Query(value = "{'username': {$regex : ?0, $options: 'i'}}")
 	Optional<Usuario> findByUsername(String username);
+
 
 	@Query(value = "{'password': {$regex : ?0, $options: 'i'}}")
 	Page<Usuario> findByPassword(String password, Pageable pageable);
@@ -50,6 +51,8 @@ public interface I_UsuarioRepository extends MongoRepository<Usuario, String> {
 
     boolean existsByUsername(String username);
 
+    boolean existsByPassword(String password);
+    
     boolean existsByEmail(String email);
 
 

src/main/java/com/api/rest/v1/security/services/UsuarioServiceImpl.java

@@ -413,6 +413,8 @@ public boolean existsByUsername(String username) {
 	}
 
 
+
+	
 	// ===========================
 	// ===== EXISTS BY EMAIL =====
 	// ===========================