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

✨ [STMT-146] 이미지 업로드를 위한 인터페이스 정의 #24

Merged
merged 4 commits into from
Feb 21, 2024
Merged

✨ [STMT-146] 이미지 업로드를 위한 인터페이스 정의 #24

merged 4 commits into from
Feb 21, 2024

Conversation

05AM
Copy link
Member

@05AM 05AM commented Feb 20, 2024

💁 해결하려는 문제를 적어주세요

  • 회원 가입 기능과 동시 개발을 위해 s3 파일 업로드 어댑터 구현 전 파일 업로드 인터페이스를 정의

🤔 어떤 방식으로 해결했는지 적어주세요

  • 파일 업로드 가능을 하는 인터페이스를 작성하였습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • 작성한 파일의 위치가 핵사고날 아키텍처에 부합하는지 고민이 됩니다.
  • 파일 관련한 crud가 한 도메인에 국한되지 않고 발생할 것으로 예상되어 file 패키지를 따로 뺐습니다.
  • 이해가 가지 않는 부분이 있다면 가감 없이 리뷰 부탁 드립니다.

🧑‍🏫 이해를 위해 필요한 자료가 있다면 첨부해주세요

file
  ㄴ application
     ㄴ port
         ㄴ in
             ㄴ FileUploadUseCase
         ㄴ out
             ㄴ FileCommandPort
     ㄴ service
         ㄴ FileUploadService
  ㄴ domain
      ㄴ FileUrl
  • FileUrl은 특별한 기능을 하지는 않지만 String 반환보다는 의미 전달을 하기 위해 만들었습니다.
  • 이후 FileCommandAdapter는 adapter - out - storage 패키지에 작성할 예정입니다. 혹시 다른 의견 있으시면 공유해주시면 감사하겠습니다!

05AM and others added 3 commits February 16, 2024 21:06
…인 api 구현 (#21)

* [STMT-151] ✨ 공통적으로 사용되는 생성 시간, 수정 시간 추가를 위한 추상 클래스 구현 (#10)

* [STMT-151] ✨ 공통적으로 사용되는 생성 시간, 수정 시간 추가를 위한 추상 클래스 구현

* [STMT-151] 🐛 설정 클래스를 명시하여 스프링 컨텍스트에서 관리하도록 설정

* [STMT-6] 👷 CI/CD를 위한 github-actions 추가

* [STMT-6] 👷 CI/CD를 위한 github-actions 추가

* [STMT-151] ✨ 로그인을 위한 멤버, OAuth 로그인 정보 모델링 (#11)

* [STMT-6] 🐛 도커 컨테이너 실행 전, 이전에 실행 중이던 컨테이너를 중지, 삭제하도록 수정 및 다수의 오타 수정

* [STMT-6] 💚 JDK 21 distribution temurin에 대한 CI/CD 자동화 테스트

* [STMT-6] 👷 CI/CD 자동화 워크플로우 완성

* [STMT-6] ✏️ 주석 제거

* [STMT-151] ✨ JWT 생성 및 검증 객체 구현 (#14)

* [STMT-151] ✨ JWT 생성 및 검증 객체 구현

* [STMT-151] ➕ JWT, OpenFeign, QueryDSL 의존성 추가

* [STMT-151] ✨ OAuth2 로그인/회원가입 기능 구현 (#16)

* [STMT-151] 🔧 QueryDSL 설정

* [STMT-151] ✨ API 공통 응답 객체 구현

* [STMT-151] ✨ 클라이언트에게 전달받은 Access Token을 이용하여 kakao에 유저 정보 요청 기능 구현

* [STMT-151] ✨ OAuth 로그인 정보를 관리하는 도메인 정의

* [STMT-151] ✨ 카카오 유저 정보 조회 응답 객체 정의

* [STMT-151] ✨ OAuth 회원가입/로그인 기능 필터 구현

* [STMT-151] ✨ OAuth 회원가입 / 로그인 인증 프로바이더 구현

* [STMT-151] ✨ JWT 인증 필터 구현

* [STMT-151] ✨ 권한 부족 및 미인증 유저가 요청시 예외발생에 대한 예외 핸들러 추가

* [STMT-151] ✨ 구현한 필터 시큐리티 필터 체인에 등록

* [STMT-151] ✨ 회원가입 추가정보 입력 API 구현

* [STMT-6] 👷 CI-CD workflow 수정 - 환경변수 설정 (#17) (#18)

- 파일 이름 변경
 - github-actions.yml -> ci-cd.yml
- gitignore에 application-secret.properties 파일 추가

Co-authored-by: zxcv9203 <41960243+zxcv9203@users.noreply.github.com>

* [STMT-6] 👷 ci-cd 워크플로우가 실패하는 원인 규명 및 문제 해결 (#19)

* [STMT-6] 👷 CI-CD workflow 수정 - 환경변수 설정 (#17)

- 파일 이름 변경
 - github-actions.yml -> ci-cd.yml
- gitignore에 application-secret.properties 파일 추가

Co-authored-by: zxcv9203 <41960243+zxcv9203@users.noreply.github.com>

* [STMT-6] ci-cd.yml event trigger 수정

* [STMT-6] ci-cd.yml 문법 오류 수정

step-Checkout에서 uses 앞 대쉬 삭제

---------

Co-authored-by: zxcv9203 <41960243+zxcv9203@users.noreply.github.com>

* [STMT-6] 🔧 개발 환경별 yml 설정 파일 작성 (#20)

Co-authored-by: zxcv9203 <41960243+zxcv9203@users.noreply.github.com>

* [STMT-6] 💚 CI/CD dev 테스트

* [STMT-6] prod 설정파일로 빌드하는 설정 추가

* [STMT-6] 💚 CICD 테스트용으로 event triger에 추가한 코드 삭제

branch `dev` 제외

* [STMT-6] 🔥 prod 설정 파일의 security logging 설정 코드 제거

* [STMT-6] 🔥 ci-cd 워크 플로우의 디버그 코드 제거

* [STMT-6] 🔥 ci-cd 워크 플로우 수정 - docker compose 배포 명령어

* [STMT-6] 🚀 docker-compose.yml 생성

---------

Co-authored-by: zxcv9203 <41960243+zxcv9203@users.noreply.github.com>
Co-authored-by: tngtied <tngtied@gmail.com>
Co-authored-by: tngtied <75424805+tngtied@users.noreply.github.com>
@05AM 05AM changed the title Stmt 146 file upload ✨ [STMT-146] 이미지 업로드를 위한 인터페이스 정의 Feb 20, 2024
@05AM 05AM added D-3 3일 전 까지 리뷰해주세요 D-2 2일 전 까지 리뷰해주세요 and removed D-3 3일 전 까지 리뷰해주세요 labels Feb 20, 2024
@zxcv9203 zxcv9203 removed the request for review from tngtied February 21, 2024 12:30
@github-actions github-actions bot added D-1 1일 전 까지 리뷰해주세요 and removed D-2 2일 전 까지 리뷰해주세요 labels Feb 21, 2024
- domain이라기 보다는 외부 dto에 가깝기 떄문에 `port`-`out`으로 위치 변경
@05AM 05AM merged commit 16ee661 into Stumeet:dev Feb 21, 2024
@05AM 05AM deleted the STMT-146-FILE-UPLOAD branch March 5, 2024 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-1 1일 전 까지 리뷰해주세요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants