-
Notifications
You must be signed in to change notification settings - Fork 24
[이지현] sprint12 #302
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
base: 이지현
Are you sure you want to change the base?
[이지현] sprint12 #302
The head ref may contain hidden characters: "\uC774\uC9C0\uD604-sprint12"
Conversation
|
지현님 죄송한데, 이전에 리뷰했던 내용도 다 변경사항에 들어와서.. 이전에 했던건 안 보이게 해주실 수 있을까요 |
|
아, 제가 10, 11 꺼 머지 하면 될까요 ? |
|
머지를 해도 마찬가지네요. base 브랜치 (codeit-bootcamp-spring:이지현)에 10, 11 때 리뷰 했던 커밋들이 들어있게해주세요. |
5c11c55 to
7901fd5
Compare
넵! 스프린트 미션 11 이후 커밋들만 남겨두었습니다! |
| /** | ||
| * SseEmitter 제거 | ||
| */ | ||
| public void remove(UUID receiverId, SseEmitter sseEmitter) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이것도 add처럼 원자성 있게 처리해야할것 같아요.
public void remove(UUID receiverId, SseEmitter sseEmitter) {
data.computeIfPresent(receiverId, (key, emitters) -> {
emitters.remove(sseEmitter);
return emitters.isEmpty() ? null : emitters;
});
}
| List<Notification> notifications = notificationRepository.findAllByReceiverId(receiverId, | ||
| Sort.by(Sort.Direction.DESC, "createdAt")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
findAll 처럼 전체 조회하는거면.. DB에서 정렬하는 것 보다, application에서 정렬하는게 DB 리소스를 덜 쓰는 일일거 같아요.
| } | ||
|
|
||
| redisTemplate.opsForList().rightPush(userKey, jwtInformation); | ||
| redisTemplate.expire(userKey, DEFAULT_TTL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 하면 userKey 하나하나에 TTL을 건게 아니라,
List에 TTL을 건거인데, 의도하신거가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 코드는 코드잇에서 제공하는 코드였습니다..!! (심화 요구사항 - 분산 환경 배포 아키텍처 구성하기)
|
캐시에서 전체를 다 읽는 코드 부분이 많아요. |
|
남겨주신 리뷰들 참고해 앞으로 수정해보겠습니다! |
기본 요구사항
웹소켓 구현하기
웹소켓 환경 구성
spring-boot-starter-websocket의존성을 추가하세요.웹소켓 메시지 브로커 설정
메모리 기반
SimpleBroker를 사용하세요./sub으로 설정하세요./pub으로 설정하세요./ws로 설정하고,SockJS연결을 지원해야 합니다.메시지 송신
첨부파일이 없는 단순 텍스트 메시지인 경우 STOMP를 통해 메시지를 전송할 수 있도록 컨트롤러를 구현하세요.
/pub/messages엔드포인트에 메시지를 전송할 수 있어야 합니다.@MessageMapping을 활용하세요.MessageCreateRequest를 그대로 활용합니다.첨부파일이 포함된 메시지는 기존의 API (
POST /api/messages)를 그대로 활용합니다.메시지 수신
클라이언트는 채널 입장 시 웹소켓으로
/sub/channels.{channelId}.messages를 구독해 메시지를 수신합니다.이를 고려해 메시지가 생성되면 해당 엔드포인트로 메시지를 보내는 컴포넌트를 구현하세요.
MessageCreatedEvent를 통해 새로운 메시지 생성 이벤트를 확인하세요.SimpMessagingTemplate를 통해 적절한 엔드포인트로 메시지를 전송하세요.SSE 구현하기
SSE 환경을 구성하세요.
클라이언트에서 SSE 연결을 위한 엔드포인트를 구현하세요.
GET /api/sse사용자별 SseEmitter 객체를 생성하고 메시지를 전송하는 컴포넌트를 구현하세요.
connect: SseEmitter 객체를 생성합니다.send,broadcast: SseEmitter 객체를 통해 이벤트를 전송합니다.cleanUp: 주기적으로 ping을 보내서 만료된SseEmitter객체를 삭제합니다.ping: 최초 연결 또는 만료 여부를 확인하기 위한 용도로 더미 이벤트를 보냅니다.SseEmitter객체를 메모리에서 저장하는 컴포넌트를 구현하세요.ConcurrentMap: 스레드 세이프한 자료구조를 사용합니다.List<SseEmitter>: 사용자 당 N개의 연결을 허용할 수 있도록 합니다. (예: 다중 탭)이벤트 유실 복원을 위해 SSE 메시지를 저장하는 컴포넌트를 구현하세요.
LastEventId를 전송해 이벤트 유실 복원이 가능하도록 해야 합니다.기존에 클라이언트에서 폴링 방식으로 주기적으로 요청하던 데이터를 SSE를 이용해 서버에서 실시간으로 전달하는 방식으로 리팩토링하세요.
새로운 알림 이벤트 전송
새 알림이 생성되었을 때 클라이언트에 이벤트를 전송하세요.
클라이언트는 이 이벤트를 수신하면 알림 목록에 알림을 추가합니다.
이벤트 명세
notifications.createdNotificationDto파일 업로드 상태 변경 이벤트 전송
파일 업로드 상태가 변경될 때 이벤트를 발송하세요.
클라이언트는 해당 상태를 수신하면 파일 상태 UI를 다시 렌더링합니다.
이벤트 명세
binaryContents.updatedBinaryContentDto채널 갱신 이벤트 전송
채널 정보가 변경될 때, 이벤트를 발송하세요.
클라이언트는 해당 이벤트를 수신하면 채널 UI를 다시 렌더링합니다.
이벤트 명세
channels.createdorupdatedordeletedChannelDto사용자 갱신 이벤트 전송
users.createdorupdatedordeletedUserDto배포 아키텍처 구성하기
다음의 다이어그램에 부합하는 배포 아키텍처를 Docker Compose를 통해 구현하세요.
Reverse Proxy/api/*,/ws/*요청은 Backend 컨테이너로 프록시 처리합니다./usr/share/nginx/html등)에 복사하세요.3000번 포트를 통해 접근할 수 있어야 합니다.BackendReverse Proxy를 통해/api/*,/ws/*요청이 이 서버로 전달됩니다.DB,Memory DB,Message BrokerBackend컨테이너가 접근 가능한 다음의 인프라 컨테이너들을 구성하세요심화 요구사항
웹소켓 인증/인가 처리하기
인증 처리
디스코드잇 클라이언트는
CONNECT프레임의 헤더에 다음과 같이Authorization토큰을 포함합니다.서버 측에서는
ChannelInterceptor를 구현하여 연결 시 토큰을 검증하고, 인증된 사용자 정보를SecurityContext에 설정해야 합니다.CONNECT프레임일 때 엑세스 토큰을 검증하는JwtAuthenticationChannelInterceptor구현체를 정의하세요.JwtAuthenticationFilter를 참고하세요.SecurityContext에 인증정보를 저장하는 대신accessor객체에 저장하세요.SecurityContextChannelInterceptor를 등록하여 이후 메시지 처리 흐름에서도 인증 정보를 활용할 수 있도록 구성하세요.인가 처리
AuthorizationChannelInterceptor를 사용해 메시지 권한 검사를 수행합니다.AuthorizationChannelInterceptor를 활용하기 위해의존성을 추가하세요.MessageMatcherDelegatingAuthorizationManager를 활용해 인가 정책을 정의하고, 채널에 추가하세요.분산 환경 배포 아키텍처 구성하기
다음의 다이어그램에 부합하는 배포 아키텍처를 Docker Compose를 통해 구현하세요.
Backend-*deploy.replicas설정을 활용하세요.Reverse Proxyupstream블록을 수정해 다음의 로드밸런싱 전략을 적용해Backend로 트래픽을 분산시켜보세요.기본값$upstream_addr변수를 활용해 실제 요청을 처리하는 서버의 IP를 헤더에 추가하고 브라우저 개발자 도구를 활용해 비교해보세요.분산환경에 따른
InMemoryJwtRegistry의 한계점을 식별하고 Redis를 활용해 리팩토링하세요.어떤 한계가 있는지 식별하고 PR에 남겨주세요.
InMemoryJwtRegistry는 JWT 토큰 정보를 각 컨테이너의 메모리에 저장하고 관리한다.backend-1에서 로그인해 토큰을 발급받어라도, 해당 정보는backend-1컨테이너에만 저장된다.backend-2)로 전달되면, 그 컨테이너의InMemoryJwtRegistry에는 해당 토큰 정보가 없으므로 토큰을 유효하지 않다고 판단한다.RedisJwtRegistry구현체를 활용하세요.분산환경에 따른 웹소켓과 SSE의 한계점을 식별하고 Kafka를 활용해 리팩토링하세요.
어떤 한계가 있는지 식별하고 PR에 남겨주세요.
일반적인 카프카 이벤트와 다르게 각 서버 인스턴스마다 이벤트를 받을 수 있어야 합니다. 따라서
컨슈머 group id를 적절히 설정하세요.주요 변경사항
1. 프로젝트 버전이 변경되었습니다.
v3.0-M123.0:api-doc버전을 따릅니다.M12: 미션 12을 의미합니다.2. 프론트엔드가 변경되었습니다.
v3.0.0스크린샷
멘토에게