Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat: EventSource를 구현하는 커스텀 EventSource 구현 #65

Merged
merged 5 commits into from
Jan 21, 2025
Merged

Feat: EventSource를 구현하는 커스텀 EventSource 구현 #65

merged 5 commits into from
Jan 21, 2025

Conversation

soyesenna
Copy link
Collaborator

1. 무슨 이유로 코드를 변경했나요?

  • SSE 구독 시 Authorization 헤더를 포함하여 보내기 위한 커스텀 EventSource를 구현했습니다

2. 어떤 위험이나 장애를 발견했나요?

  • SSE 구독 요청시 기본 EventSource객체는 커스텀하게 헤더를 설정할 수 없는 문제가 있었습니다

3. 관련 스크린샷을 첨부해주세요.

KakaoTalk_Photo_2025-01-21-10-58-51

4. 완료 사항


5. 추가 사항

@soyesenna soyesenna added the feat 신규피쳐 label Jan 21, 2025
@soyesenna soyesenna self-assigned this Jan 21, 2025
WooGi1020
WooGi1020 approved these changes Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@WooGi1020 WooGi1020 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 이건 제가 맛나게 요리해먹도록 하겠읍니다 👍

woneeeee
woneeeee approved these changes Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@woneeeee woneeeee left a comment

Choose a reason for hiding this comment

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

흐억 감사합니다! 👍🏻😁

@github-actions GitHub Actions
Copy link

구현 기능 확인하기: https://gachtaxi-h4siwo1k4-cogis-projects.vercel.app

@soyesenna soyesenna merged commit 0b68e54 into main Jan 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WooGi1020 WooGi1020 WooGi1020 approved these changes

@woneeeee woneeeee woneeeee approved these changes

Assignees

@soyesenna soyesenna

Labels
feat 신규피쳐
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@soyesenna @WooGi1020 @woneeeee