Skip to content

[Fix] SOLAPI 웹훅 배열 응답 처리 - #75

Merged
marshmallowing merged 1 commit into
developfrom
28
Aug 13, 2026
Merged

[Fix] SOLAPI 웹훅 배열 응답 처리#75
marshmallowing merged 1 commit into
developfrom
28

Conversation

@marshmallowing

@marshmallowing marshmallowing commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • 변경 사항
    • 통화 결과 웹훅이 단일 요청뿐 아니라 여러 요청을 한 번에 처리하도록 개선되었습니다.
    • 각 웹훅 결과가 순차적으로 처리되며, 직접 전달된 결과와 중첩된 결과 형식을 지원합니다.
    • 정상적으로 접수된 웹훅 요청은 HTTP 200 응답을 반환합니다.
    • 수신된 요청 건수를 확인할 수 있도록 로깅이 추가되었습니다.

@marshmallowing marshmallowing self-assigned this Aug 13, 2026
@marshmallowing marshmallowing added the 🐞 Fix 버그 수정 label Aug 13, 2026
@marshmallowing
marshmallowing merged commit cf3b076 into develop Aug 13, 2026
1 check was pending
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ca835d0e-fb26-4648-95b6-2abbeac44daf

📥 Commits

Reviewing files that changed from the base of the PR and between 0551851 and 3ed6ecd.

📒 Files selected for processing (2)
  • src/main/java/com/piuda/callcare/domain/calllog/controller/CallResultWebhookController.java
  • src/test/java/com/piuda/callcare/domain/calllog/controller/CallResultWebhookControllerTest.java

📝 Walkthrough

Walkthrough

웹훅 컨트롤러가 단일 요청 대신 요청 목록을 처리하도록 변경되었습니다. 각 결과를 순차적으로 서비스에 전달하며 수신 건수를 기록합니다. 배열 요청과 개별 결과 전달을 검증하는 테스트를 추가했습니다.

Changes

통화 결과 웹훅 일괄 처리

Layer / File(s) Summary
웹훅 목록 처리
src/main/java/com/piuda/callcare/domain/calllog/controller/CallResultWebhookController.java
receiveCallResultsList<CallResultWebhookRequest>를 받아 각 결과의 메시지 ID와 상태를 CallReminderCommandService.applyCallResult에 전달합니다. 수신 건수를 로그로 기록합니다.
일괄 처리 검증
src/test/java/com/piuda/callcare/domain/calllog/controller/CallResultWebhookControllerTest.java
배열 형태의 웹훅 요청에 HTTP 200 응답을 반환하고, 두 결과가 각각 서비스에 전달되는지 검증합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant WebhookSender
  participant CallResultWebhookController
  participant CallReminderCommandService
  WebhookSender->>CallResultWebhookController: 통화 결과 요청 목록 전송
  loop 각 웹훅 결과
    CallResultWebhookController->>CallReminderCommandService: 메시지 ID와 상태 적용
  end
  CallResultWebhookController-->>WebhookSender: HTTP 200 응답
Loading
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 28

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant