Skip to content

Conversation

@bo-ram-bo-ram
Copy link
Member

@bo-ram-bo-ram bo-ram-bo-ram commented Jan 31, 2025

Pull request

Related issue

resolve #80

Motivation and context

  • 채팅서버의 소켓 연결을 포함한 초기 셋팅을 진행했습니다.
  • 외부 브로커(kafka)와 연결 전 스프링 내부 브로커를 통해 통신을 진행했습니다.
  • 좀 더 구체화 된 다음 명세 및 문서화를 진행할 예정입니다.

Solution

  • 외부 서버와의 연동 전 기본적인 동작(하나의 서버에 접속한 유저에게 체팅 보내주기 가능) 구현하는 채팅 api를 구현했습니다.

    • 도커로 서버를 하나 더 띄워 진행하려다가 그러려면 외부 브로커가 필요한 것을 깨닫고 서버간의 통신 전 기능을 먼저 구현하려고 합니다
  • 호환성이 좋은 SockJS를 사용했습니다.

  • �Reques에선 userId를 받아오고 해당 id로 nickname을 찾는 과정에서 유저 entity, repositoy를 생성하였습니다.

    • 해당 코드는 현재 auth, workspace, chat 3개의 서버에 퍼져있는데요..!! 지난번 슬랙에서도 논의했지만 해결이 필요해보입니다 ㅠㅠ

How has this been tested

간단한 클라이언트 코드를 구현하여 진행했습니다.
스크린샷 2025-02-01 오전 3 01 20

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the docs/CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@bo-ram-bo-ram bo-ram-bo-ram added ✨ Feature 기능 추가 📦 Environment 개발 환경 세팅 🎮 BE 백엔드 무조건 스프린트내에 해야하는 것들 labels Jan 31, 2025
@bo-ram-bo-ram bo-ram-bo-ram self-assigned this Jan 31, 2025
@bo-ram-bo-ram bo-ram-bo-ram added the ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 label Jan 31, 2025
Copy link
Member

@mirlee0304 mirlee0304 left a comment

Choose a reason for hiding this comment

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

서버간 겹치는 엔티티에 대한 논의가 정말 필요할 것 같습니다 수고하셨어요!

@bo-ram-bo-ram bo-ram-bo-ram merged commit f0dffa1 into dev Feb 2, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎮 BE 백엔드 📦 Environment 개발 환경 세팅 ✨ Feature 기능 추가 ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 무조건 스프린트내에 해야하는 것들

Projects

None yet

Development

Successfully merging this pull request may close these issues.

채팅서버 초기셋팅 및 채팅 mvp api 구현

3 participants