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

[BE] 이미지 검증로직 변경 #978

Merged
merged 3 commits into from
Jan 6, 2025
Merged

Conversation

pilyang
Copy link
Collaborator

@pilyang pilyang commented Dec 29, 2024

이슈번호

#969 사전작업, #866

PR 내용

이미지 검증로직 위치 변경

  • api에서 요청주는 이미지 수 spring validation 사용해서 request body 에서 @Size 사용 (기존 서비스 -> 컨트롤러 검증할 수 있도록 변경)
  • 이미지 사이즈, 확장자 검증 로직 위치 변경 : filesystem pakcage에 이미지 검증을 위한 validation 서비스 추가
    • 기존 공지와 스레드의 중복코드 제거 -> 필요한 서비스에서 검증서비스 추가해서 검증할 수 있도록

참고자료

의논할 거리

서비스 위치 적절한가 정도...? 확인 필요할수도...

Copy link

Test Results

159 files  159 suites   31s ⏱️
509 tests 509 ✅ 0 💤 0 ❌
513 runs  513 ✅ 0 💤 0 ❌

Results for commit 2a4e4d4.

Copy link
Collaborator

@SproutMJ SproutMJ left a comment

Choose a reason for hiding this comment

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

공통으로 묶어서 서비스로직으로 만든거 잘 봤습니다!

@pilyang pilyang merged commit 9e795ba into develop Jan 6, 2025
3 checks passed
@pilyang pilyang deleted the refactor/be/image-validation branch January 6, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants