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] refactor/#552 Image 패키징 리팩터링 #534

Closed
wants to merge 7 commits into from

Conversation

junpakPark
Copy link
Collaborator

@junpakPark junpakPark commented Oct 1, 2023

작업 대상

  • Image 패키지
  • Image 패키지를 의존하는 Service 리팩터링

📄 작업 내용

Image 패키지 내부 리팩터링

  • Image 패키지 레이어 재분배
  • Image 클래스 common에서 Image 패키지로 이동
  • 불필요 의존성 제거
  • domain 리팩터링
  • 관련 커스텀 예외 구현
  • Infrastructure 리팩터링
    • tempFile 대신 InputStream을 이용한 방식으로 변경
    • try-with-resource 적용
  • 불필요 클래스 삭제

Image 패키지 의존 Service 리팩터링

  • 스프링 이벤트를 이용하여, S3 Upload 과정과 DB Transaction 과정 분리
  • 패키지별 Info 수정 시, 사진 변경 기능 구현

🙋🏻 주의 사항

  • 동시성 이슈 예방을 위한 파일 이름 변경
    yyyyMMddHHmmssssss.파일형식 -> yyMMddHHmmss_UUID 일부문자열.파일형식

스크린샷

📎 관련 이슈

closed #552

레퍼런스

@junpakPark junpakPark added BE 백엔드 관련 이슈 우선순위 : 하 refactor 리팩토링 관련 이슈 labels Oct 1, 2023
@junpakPark junpakPark self-assigned this Oct 1, 2023
@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Unit Test Results

0 files   -   67  0 suites   - 67   0s ⏱️ -24s
0 tests  - 288  0 ✔️  - 288  0 💤 ±0  0 ±0 
0 runs   - 299  0 ✔️  - 299  0 💤 ±0  0 ±0 

Results for commit 9c307f9. ± Comparison against base commit 42b5813.

@junpakPark junpakPark changed the title [BE] refactor/#533 Image 패키징 리팩터링 [BE] refactor/#552 Image 패키징 리팩터링 Oct 5, 2023
@junpakPark junpakPark closed this Oct 31, 2023
@semnil5202 semnil5202 deleted the refactor/image branch February 9, 2024 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 하
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant