Skip to content

[#52] 로드 밸런싱 환경에서의 채팅 기능 동작을 위한 코드 수정 #54

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
Merged
merged 19 commits into from
Jun 23, 2023
Merged

[#52] 로드 밸런싱 환경에서의 채팅 기능 동작을 위한 코드 수정 #54

merged 19 commits into from
Jun 23, 2023

Conversation

@tilsong
Copy link
Collaborator

@tilsong tilsong commented May 16, 2023

PR 유형

  • 버그 수정
  • 기능
  • 코드 스타일
  • 리팩토링
  • 빌드
  • CI/CD
  • 문서
  • 기타

변경사항

변경 전

  • 로드 밸런싱 환경에서 채팅 기능 정상 작동x

변경 후

  • 로드 밸런싱 환경에서도 채팅이 정상적으로 동작하도록 기능을 수정했습니다.
    • 이를 위해 Redis의 pub/sub을 적용했습니다.
    • 채팅 관련 테스트를 수정, 추가했습니다.

사용 방법

  • 로컬 테스트를 위해 로컬 환경에 Redis 설치 및 운용이 필요합니다.

기타

  • 배포 서버에서도 Redis 설치 및 운용이 필요합니다.
  • Redis 연결 및 객체의 직렬화를 위해 다음의 의존성을 추가했습니다.
    • org.springframework.boot:spring-boot-starter-data-redis
    • com.fasterxml.jackson.datatype:jackson-datatype-jsr310
  • 실제 어플리케이션과 레디스와의 연동은 포스트맨을 통해 테스트했습니다.

tilsong added 17 commits May 14, 2023 16:03
@tilsong
[#52] 채팅 도메인의 레포지토리 단위 테스트 작성
8f759dc
@tilsong
[#52] 채팅 도메인 객체의 메소드 테스트 작성
f271789
@tilsong
[#52] Chat 도메인 서비스 생성 및 관련 테스트 코드 작성
16df7fa 
- 기존 ChatController에 있던 로직을 ChatService를 생성하여 이전시켰습니다.

- Chat 서비스의 테스트를 위한 환경 클래스를 생성하고, 서비스의 메서드를 테스트하는 코드를 작성했습니다.

- 테스트 중 관련 로직을 개선하고, dto를 수정했습니다.
@tilsong
[#52] RoomController 오타 및 dto validate 수정, Question 버그 제거
d476faf
@tilsong
[#52] ChatController, ChatService 메서드 명 수정 및 Chat 엔티티의 BaseEntity 상속 제거
6496664
@tilsong
[#52] ChatService 메서드 리팩토링 및 dto 코드 수정
64f365f
@tilsong
[#52] Chat, Room 엔티티 연관 관계 제거
f93c887
@tilsong
[#52] Chat 엔티티 변경으로 인한 코드 수정
a415288
@tilsong
[#52] Chat 엔티티 변경으로 인한 코드 수정2
3f3a8d0
@tilsong
[#52] 레디스 연결 관련 의존성 및 설정 추가
47f830a 
- 추가된 의존성
  - spring-boot-starter-data-redis
  - com.fasterxml.jackson.datatype:jackson-datatype-jsr310
@tilsong
[#52] RoomRepository 메소드 추가
3767e10
@tilsong
[#52] ObjectMapperUtils 클래스 테스트 작성
967f64c
@tilsong
[#52] Chat 엔티티의 연관 관계 제거 및 관련 로직 변경
9e5cd39
@tilsong
[#52] Redis pub/sub 연동
19aacd1 
-  채팅방에 대한 sse 연결이 현재 존재할 경우 redis를 통해 채팅을 전달하도록 코드 작성
@tilsong
[#52] 변경된 코드들에 대한 테스트 코드 변경, 오타 수정
6b98254
@tilsong
[#52] RoomService 코드 개선
1c316f1
@tilsong
[#52] LiveChatManager 코드 개선
6278377
@tilsong tilsong requested a review from f-lab-fi May 16, 2023 10:59
@tilsong tilsong self-assigned this May 16, 2023
@tilsong tilsong linked an issue May 16, 2023 that may be closed by this pull request
로드 밸런싱 환경에서의 채팅 기능 동작을 위한 코드 수정 #52
Closed
2 tasks
@tilsong tilsong merged commit 9003d87 into main Jun 23, 2023
tilsong added a commit that referenced this pull request Jul 16, 2023
@tilsong
[#54] JPA 엔티티 간 양방향 관계 때문에 발생하는 순환 참조 문제 해결
484121d 
- @JsonIgnore를 통해 순환 참조가 발생하는 필드를 Json으로 직렬화하는 과정에서 무시하도록 함.
tilsong added a commit that referenced this pull request Jul 17, 2023
@tilsong
[#54] Spring Security Cors 에러 해결
e8c9ade
tilsong added a commit that referenced this pull request Jul 17, 2023
@tilsong
Merge pull request #57 from f-lab-edu/feature/54
b39fb80 
[#54] 사이트 URL 적용 및 SSL 처리
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@f-lab-fi f-lab-fi Awaiting requested review from f-lab-fi

Assignees

@tilsong tilsong

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

로드 밸런싱 환경에서의 채팅 기능 동작을 위한 코드 수정

2 participants

@tilsong