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

fix: privateAlert channelService에서 하기, 소켓 끊길 때 db 업데이트 잘하기, s3 이미지 api 추가 #114

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

tomatozil
Copy link
Contributor

  1. privateAlert에서 던져지는 에러가 문제를 일으켰습니다
  • 그래서 ChannelGateway에서 초대한 유저와 받은 유저가 유효한지를 검사하는게 맞나? 의문이 들어서 privateAlert 함수의 로직을 ChannelService로 옮겼읍니다.
  1. 채널 소켓이 disconnect될 때 db를 업데이트하지 못하는 문제가 있었습니다.
  • 예전에 발견했던 문제입니다 handleDisconnect에 WsAuthGuard가 적용되지 않습니다 그래서 연결시 socketIoAdapter에서 넣어줬던 user 객체를 그대로 씁니다. user의 channelSocketId가 null이어서 if (!user || client.id !== user.channelSocketId) 조건문에 걸리는 것이었습니다. 진짜 의도는 이게 이미 끊긴 client인지 거르는 것이라 생각해서 단순하게 if (!user || !client.id)로 바꿨습니다.
  1. 외부의 s3 버킷 접근을 위한 preSignedUrl 을 발급해주는 api들을 추가했습니다.

@tomatozil tomatozil merged commit 01bcefb into main Apr 23, 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.

1 participant