Skip to content

Comments

[KW-682] feat/webhook handler#9

Merged
willjsw merged 12 commits intodevelopfrom
KW-682/feat/webhook-handler
Jun 16, 2025
Merged

[KW-682] feat/webhook handler#9
willjsw merged 12 commits intodevelopfrom
KW-682/feat/webhook-handler

Conversation

@willjsw
Copy link
Contributor

@willjsw willjsw commented Jun 16, 2025

🔷 Jira Ticket ID

KW-682


📌 작업 내용 및 특이사항

  • Connection 상태 업데이트 / DID Key 생성 및 등록 / VC 발급을 위한 ACApy Webhook 핸들러 구현

📚 참고사항

@github-actions github-actions bot changed the title Kw 682/feat/webhook handler [KW-682] Kw 682/feat/webhook handler Jun 16, 2025
@willjsw willjsw self-assigned this Jun 16, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a webhook handler for ACA-Py to manage connection state updates, DID creation/registration, and VC issuance.

  • Implements AcapyWebhookService with handlers for connection, credential, and out-of-band webhooks
  • Adds new DTOs for ACA-Py request/response payloads
  • Updates controller and agent classes to route and log incoming webhooks

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
service/AcapyWebhookService.java New webhook processing flows for connection, VC, OOB
controller/AcapyWebhookController.java Endpoints wired to webhook service with logging
dto/request & dto/response packages Records for mediator and hospital create/issue flows
agent/HospitalAgent.java & agent/MediatorAgent.java Calls to ACA-Py endpoints for invitations, DIDs, VCs
dto/request/DidCreateRequest.java & response/DidCreateResponse.java Deleted legacy DID DTOs
Comments suppressed due to low confidence (1)

src/main/java/com/doubleo/didagent/agent/MediatorAgent.java:33

  • [nitpick] The error message references 'MemberConnection fetch' but this method creates a mediator invitation; update the message for clarity.
log.error(
                                    "Mediator MemberConnection fetch error: {}",
                                    error.getMessage());

willjsw and others added 7 commits June 16, 2025 15:15
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
….java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
….java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

@midday2612 midday2612 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P5: 확인했습니다. 고생하셨어요....

Copy link
Contributor

@coffeesigma coffeesigma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

장인정신이 보이는 코드네요...
다 만드느라 고생하셨습니다

@willjsw willjsw merged commit 6228a54 into develop Jun 16, 2025
1 check passed
@willjsw willjsw changed the title [KW-682] Kw 682/feat/webhook handler [KW-682] feat/webhook handler Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants