feat(logging): add Loggable annotation and LoggingMethodAspect for method execution logging

Author: mjabascal10

backend/src/main/java/com/park/utmstack/aop/logging/Loggable.java

@@ -0,0 +1,13 @@
+package com.park.utmstack.aop.logging;
+
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
+@Target(ElementType.METHOD)
+@Retention(RetentionPolicy.RUNTIME)
+public @interface Loggable {
+}
+
+

backend/src/main/java/com/park/utmstack/aop/logging/impl/LoggingMethodAspect.java

@@ -0,0 +1,30 @@
+package com.park.utmstack.aop.logging.impl;
+
+import lombok.extern.slf4j.Slf4j;
+import org.aspectj.lang.ProceedingJoinPoint;
+import org.aspectj.lang.annotation.Around;
+import org.aspectj.lang.annotation.Aspect;
+import org.slf4j.MDC;
+import org.springframework.stereotype.Component;
+
+@Aspect
+@Component
+@Slf4j
+public class LoggingMethodAspect {
+    @Around("@annotation(com.park.utmstack.aop.logging.Loggable)")
+    public Object logExecution(ProceedingJoinPoint joinPoint) throws Throwable {
+        String traceId = MDC.get("traceId");
+        String methodName = joinPoint.getSignature().toShortString();
+        long start = System.currentTimeMillis();
+
+        try {
+            Object result = joinPoint.proceed();
+            long duration = System.currentTimeMillis() - start;
+            log.debug("[{}] Method {} executed successfully in {}ms", traceId, methodName, duration);
+            return result;
+        } catch (Exception ex) {
+            log.error("[{}] Method {} failed: {}", traceId, methodName, ex.getMessage(), ex);
+            throw ex;
+        }
+    }
+}

backend/src/main/java/com/park/utmstack/service/tfa/TotpTfaService.java

@@ -4,6 +4,7 @@
 import com.google.zxing.MultiFormatWriter;
 import com.google.zxing.client.j2se.MatrixToImageWriter;
 import com.google.zxing.common.BitMatrix;
+import com.park.utmstack.aop.logging.Loggable;
 import com.park.utmstack.config.Constants;
 import com.park.utmstack.domain.User;
 import com.park.utmstack.domain.tfa.TfaMethod;
@@ -58,6 +59,7 @@ public TfaInitResponse initiateSetup(User user) {
     }
 
     @Override
+    @Loggable
     public TfaVerifyResponse verifyCode(User user, String code) {
         TfaSetupState tfaSetupState = cache.getState(user.getLogin(), TfaMethod.TOTP)
                 .orElseThrow(() -> new IllegalStateException("No TFA setup found for user: " + user.getLogin()));