Skip to content

Conversation

@adkm12
Copy link
Collaborator

@adkm12 adkm12 commented Dec 2, 2024

작업 내용

  • POST api/ai/audio/:mindmapId 추가
  • nginx BODY 최대 사이즈 변경
  • 소켓 aiPending 요청 추가
  • 파일 확장자 validation 파이프 추가

추가 내용

  • redis에 연결 정보가 없거나 잘못된 연결ID로 연결시 connectionId로 redis 데이터 캐싱

논의하고 싶은 내용

- redis에 연결 정보가 없거나 잘못된 연결ID로 연결시 connectionId로 redis 데이터 캐싱
- POST api/ai/audio/:mindmapId 추가
- nginx BODY 최대 사이즈 변경
- 소켓 aiPending 요청 추가
- 파일 확장자 validation 파이프 추가
@adkm12 adkm12 added the ⚡️ BE 백엔드 관련 feature를 다룹니다 label Dec 2, 2024
@adkm12 adkm12 requested a review from kimnamheeee December 2, 2024 14:51
@adkm12 adkm12 self-assigned this Dec 2, 2024
}

@Get(':connectionId')
@UseGuards(AuthGuard('jwt'))
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기서 혹시 비회원이면 어떤 코드로 에러가 오게 되나요?

Comment on lines 44 to 46
if (!role) {
throw new UnauthorizedException('권한이 없습니다.');
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 403으로 보내 주실 수 있나요

Copy link
Collaborator

@kimnamheeee kimnamheeee left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ~ ~ ~ status 코드만 한 번 검토해 주세용

@adkm12 adkm12 merged commit a97f44b into dev Dec 2, 2024
@adkm12 adkm12 deleted the feature-2-be branch December 2, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚡️ BE 백엔드 관련 feature를 다룹니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants