Skip to content

Conversation

blu3fishez and others added 30 commits November 25, 2024 20:53
- 분리하면서 아예 공식 모듈로 승격
- 스터디 세션, 시그널링 서버 핸들러보다 더 상위의 기초적인 서버 설정과 서버 인스턴스를 맡는 전역 게이트웨이
- 길어지는 로직을 아예 파일로 분리했음
- 현재로써는 로직의 수가 많지 않기에 가독성 측면에서 분리가 좋다고 판단했습니다.
- `backend/src/room/services/room-create.service.ts`

해당 파일에서 새롭게 `redisClient` 를 받아와서 새로운 roomId 값을 생성하도록 개선
- 이제 redis-stack을 사용
[Fix] 세션 생성 오류 수정, 도커 컴포즈 설정파일 업데이트, 그 외 오류 수정
- 설정 파일이 존재하지않으므로 설정하지 않도록 변경
- .gitignore 에 맞게 컨테이너 폴더 이름 변경
-  마이페이지에 있던 getUserInfo 로직을 useAuth로 이동
- CSRF 공격 방지를 위한 프로덕션 빌드에서의 도메인 설정이 가능하도록 수정
[Refactor] index.html 메타데이터 og-image 확장자 변경 및 deprecated 태그 제거
[Fix] select 컴포넌트 관련 오류 해결
[Fix] websocket gateway CORS 허용
[Refactor] WebSocket 이벤트명 변경
ShipFriend0516 and others added 20 commits November 26, 2024 23:06
- connectionList => connectionMap
- connectionMap 에서 자신의 연결정보 삭제하고 전송
Co-authored-by: blu3piece <kcw2205@naver.com>
Co-authored-by: blu3piece <kcw2205@naver.com>
[Refactor] 서버 세션 리팩토링 대응하기
[Feat] 질문지 스크랩 기능(조회, 스크랩, 스크랩 취소) 구현
@blu3fishez blu3fishez self-assigned this Nov 27, 2024
@blu3fishez blu3fishez changed the title [배포] 5주차 작업 내용 중간 배포 [Deploy] 5주차 작업 내용 중간 배포 Nov 27, 2024
@blu3fishez blu3fishez changed the title [Deploy] 5주차 작업 내용 중간 배포 [Deploy] 5주차 스프린트 배포 - 2 Nov 27, 2024
Copy link
Collaborator

@twalla26 twalla26 left a comment

Choose a reason for hiding this comment

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

LGTM~!!! 👍

Copy link
Member

@ShipFriend0516 ShipFriend0516 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
Member

@yiseungyun yiseungyun left a comment

Choose a reason for hiding this comment

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

MZTM 👍

@blu3fishez blu3fishez merged commit 625e190 into main Nov 27, 2024
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.

5 participants