Logging the name for create-requests for already existent users
[demos/kafka/outbox] / src / main / java / de / juplo / boot / data / jdbc / UserController.java
index 3087290..0a4a62f 100644 (file)
@@ -3,12 +3,17 @@ package de.juplo.boot.data.jdbc;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.context.ApplicationEventPublisher;
+import org.springframework.dao.IncorrectResultSizeDataAccessException;
 import org.springframework.http.ResponseEntity;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.StreamUtils;
 import org.springframework.web.bind.annotation.*;
 import org.springframework.web.servlet.support.ServletUriComponentsBuilder;
 import org.springframework.web.util.UriComponents;
 
+import javax.servlet.http.HttpServletRequest;
+import java.io.IOException;
+import java.nio.charset.Charset;
 import java.time.LocalDateTime;
 
 import static de.juplo.boot.data.jdbc.UserEvent.Type.CREATED;
@@ -41,8 +46,12 @@ public class UserController {
             @RequestBody String username) {
         String sanitizedUsername = UserController.sanitize(username);
         User user = new User(sanitizedUsername, LocalDateTime.now(), false);
+
+        // Triggering a unique-error for username prevents persistence
         repository.save(user);
         publisher.publishEvent(new UserEvent(this, CREATED, sanitizedUsername));
+        user = repository.findByUsername(sanitizedUsername);
+
         UriComponents uri =
             builder
                 .fromCurrentRequest()
@@ -86,4 +95,22 @@ public class UserController {
 
         return string.trim().toLowerCase();
     }
+
+    @ExceptionHandler
+    public ResponseEntity<?> incorrectResultSizeDataAccessException(
+        HttpServletRequest request,
+        IncorrectResultSizeDataAccessException e
+        )
+    {
+      String username;
+      try {
+          username = StreamUtils.copyToString(request.getInputStream(), Charset.defaultCharset());
+      }
+      catch (IOException ioe)
+      {
+        username = e.getMessage() + " -> " + ioe.getMessage();
+      }
+      LOG.info("User {} already exists!", username);
+      return ResponseEntity.badRequest().build();
+    }
 }