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.

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

더 많은 브라우저에서 WebSocket 연결 문제가 없도록 수정 #207

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

hoqn
Copy link
Collaborator

@hoqn hoqn commented Dec 3, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

구버전 브라우저 환경에서도 WebSocket 연결 문제가 없도록 수정했습니다.

✅ 작업 내용

  • 과거 버전에서도 http/https, 그리고 상대 url을 웹소켓 주소로 제대로 해석할 수 있도록 polyfill을 작성했습니다.
  • polyfill은 단순히 monkey patch로 해주었습니다.

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

문제가 나타날 수 있는 환경을 위해 Firefox 121 버전을 설치 후 테스트해보았습니다.

(좌측: 기존, 우측: 개선)

스크린샷 2024-12-03 오후 10 30 01

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

@hoqn hoqn added the 🐞 fix label Dec 3, 2024
@hoqn hoqn self-assigned this Dec 3, 2024
@hoqn hoqn requested a review from a team as a code owner December 3, 2024 13:41
@hoqn hoqn requested review from parkblo and heegenie and removed request for a team December 3, 2024 13:41
Copy link
Collaborator

@parkblo parkblo left a comment

Choose a reason for hiding this comment

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

호균님 고생많으셨습니다! 최근 정신없으실텐데 픽스 작업해주셔서 감사합니다! 🙌
몽키 패치로 간결하게 프로토콜을 변경할 수 있는게 신기하네요 ㅎㅎ 👍

Copy link
Collaborator

@heegenie heegenie left a comment

Choose a reason for hiding this comment

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

피곤하셨을텐데 늦은 시간까지 작업해주셔서 감사합니다!
몽키 패치로 해결하신 게 신기하네요 ㅎㅎ

@heegenie heegenie merged commit 90b2d42 into dev Dec 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

구버전 브라우저 환경에서 Websocket 연결이 안 되는 문제
3 participants