Skip to content

Commit ac381c3

Browse files
committed
feat: 컨트롤러 로직 수정
1 parent 069877a commit ac381c3

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

src/main/java/com/threestar/trainus/domain/payment/controller/PaymentController.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,13 +50,15 @@ public ResponseEntity<BaseResponse<Void>> verifyAmount(HttpSession session,
5050
@RequestBody SaveAmountRequest request) {
5151
Integer amount = (Integer)session.getAttribute(request.getOrderId());
5252
log.info("amount = {}", amount);
53-
if (amount == null || !amount.equals(request.getAmount())) {
54-
//todo 세션 삭제 안했는데 괜찮은지
55-
return BaseResponse.error("결제 금액 정보가 유효하지 않습니다", null, HttpStatus.BAD_REQUEST);
56-
}
5753

58-
session.removeAttribute(request.getOrderId());
59-
return BaseResponse.ok("Payment is valid", null, HttpStatus.OK);
54+
try {
55+
if (amount == null || !amount.equals(request.getAmount())) {
56+
return BaseResponse.error("결제 금액 정보가 유효하지 않습니다", null, HttpStatus.BAD_REQUEST);
57+
}
58+
return BaseResponse.ok("Payment is valid", null, HttpStatus.OK);
59+
} finally {
60+
session.removeAttribute(request.getOrderId());
61+
}
6062
}
6163

6264
@PostMapping("/confirm")

0 commit comments

Comments
 (0)