Skip to content

feat: Add Header argument resolver#5

Merged
NARUBROWN merged 7 commits intoNARUBROWN:mainfrom
Scanf-s:feat/header-resolver-implementation
Jan 26, 2026
Merged

feat: Add Header argument resolver#5
NARUBROWN merged 7 commits intoNARUBROWN:mainfrom
Scanf-s:feat/header-resolver-implementation

Conversation

@Scanf-s
Copy link
Copy Markdown
Contributor

@Scanf-s Scanf-s commented Jan 25, 2026

Summary

Resolves #3
Implemented an HTTP argument resolver to read headers from requests within controllers.
This allows users to access HTTP headers and process business logic based on header values.

Tests

  • Ran go fmt
  • Validated implementation with unit tests in header_resolver_test.go
image
  • Verified with demo test via a demo controller that returns header information.

Removed Kafka and RabbitMQ options to run demo locally

image

Scanf-s and others added 5 commits January 25, 2026 21:18
- 이벤트 기반 실행을 위한 ConsumerRuntime 추가
- HTTP 컨텍스트와 분리된 ConsumerRequestContext 도입
- 컨슈머 전용 ArgumentResolver 구현 (EventName, Payload, DTO, EventBus)
- Registry + Invoker 기반의 컨트롤러 스타일 이벤트 핸들러 지원
- HTTP와 이벤트 컨슈머 간 실행 모델 완전 분리 보장
- RabbitMQ 기반 이벤트 Reader / Writer 구현
- Read / Write 옵션 분리 및 부트스트랩 연동
- ConsumerRuntime에서 RabbitMQ 이벤트 소비 지원
- default exchange 사용 금지 및 명시적 exchange 정책 적용
- Kafka와 동일한 이벤트 실행 모델로 통합
@Scanf-s Scanf-s requested a review from NARUBROWN as a code owner January 25, 2026 12:40
Comment thread internal/event/consumer/invoker.go Outdated
Comment thread internal/resolver/header_resolver_test.go Outdated
Copy link
Copy Markdown
Owner

@NARUBROWN NARUBROWN left a comment

Choose a reason for hiding this comment

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

안녕하세요 기여자님,

이슈에서 제안해주신대로 잘 구현되어 있음을 확인했습니다.
바로 머지하도록 하겠습니다 Spine에 기여해주셔서 감사합니다.

@NARUBROWN NARUBROWN merged commit 2cb3d77 into NARUBROWN:main Jan 26, 2026
@Scanf-s Scanf-s deleted the feat/header-resolver-implementation branch January 26, 2026 05:47
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.

Add Header Argument Resolver

2 participants