Release 1.0.2 (#10)

## 📝 작업 내용

- feature: 메뉴 옵션 그룹 생성 시 메뉴 옵션 그룹 내 중복 옵션 이름이 있는지 검사
- feature: 매장의 전체 카테고리 & 메뉴 조회 시 숨김(HIDE) 상태의 메뉴는 조회되지 않도록 수정
- feature: 주문 결제 도메인 KSCAT 현금 영수증 결제 지원 (승인 번호, 거래 일시, 거래 고유 번호)
- feature: 개발 서버 디버그 로그 `notification` 패키지 -> 전체 패키지 출력하도록 변경
- fix: 토큰 갱신 시 분산락이 적용되지 않는 현상 수정
- fix: sse 이벤트 데이터 중 id값 문자열로 변환하여 이벤트 전송 (프론트엔드 bigint 편의)
- docs: swagger 개발 서버 도메인 추가
- docs: 주문 결제 도메인 모델 문서 업데이트

## 📷 스크린샷 (선택)

## 💬 리뷰 요구사항(선택)

Author: handwoong

docker-compose.yml

@@ -2,7 +2,7 @@ services:
   everyonewaiter-api:
     container_name: everyonewaiter-api
     build: .
-    image: everyonewaiter-api:1.0.1
+    image: everyonewaiter-api:1.0.2
     ports:
       - "8081:8081"
     volumes:

gradle.properties

@@ -1,6 +1,6 @@
 # Artifact
 appGroup=com.everyonewaiter
-appVersion=1.0.1
+appVersion=1.0.2
 
 # Plugin
 springBoot=3.5.4

src/main/java/com/everyonewaiter/adapter/web/api/device/MenuApiSpecification.java

@@ -13,7 +13,11 @@
 interface MenuApiSpecification {
 
   @SecurityRequirements
-  @Operation(summary = "매장 메뉴 카테고리 및 메뉴 목록 조회", description = "매장 메뉴 카테고리 및 메뉴 목록 조회 API")
+  @Operation(
+      summary = "매장 메뉴 카테고리 및 메뉴 목록 조회",
+      description = "매장 메뉴 카테고리 및 메뉴 목록 조회 API<br/><br/>" +
+          "숨김(HIDE) 상태의 메뉴는 조회되지 않습니다."
+  )
   @ApiResponse(responseCode = "200", description = "매장 메뉴 카테고리 및 메뉴 목록 조회 성공")
   @ApiErrorResponse(
       summary = "매장 메뉴 카테고리 및 메뉴 목록 조회 실패",

src/main/java/com/everyonewaiter/adapter/web/config/AuthenticationDeviceResolver.java

@@ -3,6 +3,7 @@
 import static com.everyonewaiter.adapter.web.HttpRequestParser.parseRequestUri;
 import static java.util.Arrays.stream;
 import static java.util.Objects.requireNonNull;
+import static org.slf4j.LoggerFactory.getLogger;
 
 import com.everyonewaiter.application.auth.required.SignatureEncoder;
 import com.everyonewaiter.application.device.provided.DeviceFinder;
@@ -12,9 +13,11 @@
 import com.everyonewaiter.domain.shared.AuthenticationException;
 import jakarta.servlet.http.HttpServletRequest;
 import java.time.Duration;
+import java.util.Arrays;
 import lombok.Data;
 import lombok.NonNull;
 import lombok.RequiredArgsConstructor;
+import org.slf4j.Logger;
 import org.springframework.core.MethodParameter;
 import org.springframework.http.HttpMethod;
 import org.springframework.stereotype.Component;
@@ -27,6 +30,8 @@
 @RequiredArgsConstructor
 class AuthenticationDeviceResolver implements HandlerMethodArgumentResolver {
 
+  private static final Logger LOGGER = getLogger(AuthenticationDeviceResolver.class);
+
   private static final String ACCESS_KEY = "x-ew-access-key";
   private static final String SIGNATURE = "x-ew-signature";
   private static final String TIMESTAMP = "x-ew-timestamp";
@@ -61,6 +66,8 @@ public Device resolveArgument(
           );
 
           if (!device.isActive() || stream(annotation.purpose()).noneMatch(device::hasPurpose)) {
+            LOGGER.debug("Device not have require purpose. have: {}, required: {}",
+                device.getPurpose(), Arrays.toString(annotation.purpose()));
             throw new AccessDeniedException();
           }
 
@@ -74,6 +81,7 @@ private void validateSignature(Device device, RequestSignature requestSignature)
     String plainText = requestSignature.plainText(device);
 
     if (!signatureEncoder.matches(signature, plainText, device.getSecretKey())) {
+      LOGGER.debug("Device signature not match. plain: {}, signature: {}", plainText, signature);
       throw new AuthenticationException();
     }
   }
@@ -98,11 +106,14 @@ public RequestSignature(HttpServletRequest request) {
         throw new AuthenticationException();
       }
 
+      long validDuration = Duration.ofMinutes(5).toMillis();
       long currentTime = System.currentTimeMillis();
-      long maxTime = currentTime + Duration.ofMinutes(5).toMillis();
-      long minTime = currentTime - Duration.ofMinutes(5).toMillis();
+      long maxTime = currentTime + validDuration;
+      long minTime = currentTime - validDuration;
 
       if (timestamp < minTime || timestamp > maxTime) {
+        LOGGER.debug("Device timestamp duration is invalid. timestamp: {}, max: {}, min: {}",
+            timestamp, maxTime, minTime);
         throw new AuthenticationException();
       }
     }

src/main/java/com/everyonewaiter/adapter/web/docs/SwaggerConfiguration.java

@@ -164,6 +164,7 @@ private Info info() {
 
   private List<Server> servers() {
     return List.of(
+        new Server().url("https://devapi.everyonewaiter.com").description("Development"),
         new Server().url("http://localhost:8081").description("Local")
     );
   }

src/main/java/com/everyonewaiter/application/auth/SignInTokenService.java

@@ -41,7 +41,7 @@ public SignInToken createToken(Long accountId) {
   }
 
   @Override
-  @DistributedLock(key = "#refreshToken")
+  @DistributedLock(key = "#signInTokenRenewRequest.refreshToken")
   public Optional<SignInToken> renewToken(SignInTokenRenewRequest signInTokenRenewRequest) {
     JwtPayload payload = jwtProvider.decode(signInTokenRenewRequest.refreshToken())
         .orElseThrow(AuthenticationException::new);

src/main/java/com/everyonewaiter/application/menu/MenuModifyService.java

@@ -141,7 +141,7 @@ public void deleteAll(Long storeId, MenuDeleteRequest deleteRequest) {
       menuRepository.deleteAll(menus);
 
       var event1 = new MenuImageDeleteEvent(menus.stream().map(Menu::getImage).toList());
-      var event2 = new SseEvent(storeId, MENU, DELETE, deleteRequest.menuIds());
+      var event2 = new SseEvent(storeId, MENU, DELETE, deleteRequest.getStringMenuIds());
 
       applicationEventPublisher.publishEvent(event1);
       applicationEventPublisher.publishEvent(event2);

src/main/java/com/everyonewaiter/application/sse/SseSendEventHandler.java

@@ -16,7 +16,7 @@ class SseSendEventHandler {
   @Async("eventTaskExecutor")
   @TransactionalEventListener
   public void handle(SseEvent event) {
-    sseSender.send(event.storeId().toString(), event);
+    sseSender.send(event.storeId(), event);
   }
 
 }

src/main/java/com/everyonewaiter/domain/AggregateRootEntity.java

@@ -47,6 +47,10 @@ void markNotNew() {
     this.isNew = false;
   }
 
+  protected String getStringId() {
+    return String.valueOf(getId());
+  }
+
   @Override
   @SuppressWarnings("java:S2097") // Add a type test to this method.
   public final boolean equals(Object o) {

src/main/java/com/everyonewaiter/domain/device/Device.java

@@ -136,11 +136,11 @@ private void update(
     this.tableNo = tableNo;
     this.paymentType = requireNonNull(paymentType);
 
-    registerEvent(new SseEvent(store.getId(), DEVICE, UPDATE, getId()));
+    registerEvent(new SseEvent(getStoreId(), DEVICE, UPDATE, getStringId()));
   }
 
   public void delete() {
-    registerEvent(new SseEvent(store.getId(), DEVICE, DELETE, getId()));
+    registerEvent(new SseEvent(getStoreId(), DEVICE, DELETE, getStringId()));
   }
 
   public Long getStoreId() {