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.

Sign up for GitHub

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

Jump to bottom

충남대 BE_윤정훈 6주차 과제 (1단계) #146

Merged
merged 58 commits into from
Aug 1, 2024
Merged

충남대 BE_윤정훈 6주차 과제 (1단계) #146

merged 58 commits into from
Aug 1, 2024

Conversation

yunjunghun0116
Copy link

@yunjunghun0116 yunjunghun0116 commented Jul 31, 2024 •
edited
Loading

과제 수행 과정

  1. 팀원들과 API URL 통일
  2. 팀원들과 Request, Response DTO 통일
  3. 주문 완료 후 메시지를 보내는 과정에서 발생한 오류는 401 상태코드를 보내고, Redirect는 하지않도록 변경함으로써 기능의 의도를 명확히 함

팀원들과 함께 작성한 명세서를 바탕으로 배포된 주소입니다!

API명세서-Swagger 보러가기

코드 리뷰 중점 사항

  • RESTful 하게 작성되었는지?
  • 작성된 코드보다 더 적절한 상태코드가 존재하는지?
    를 위주로 리뷰 부탁드립니다!

+@

  • 월요일에 진행한 수업(배포 및 인프라)과정을 학습하며 배포과정을 진행하다보니 1단계를 수행하면서 2단계 중 일부인 배포하기를 진행했습니다
  • 그래서 우선은 1단계 과제인 API 명세를 통일하고, 변경사항을 반영하는것을 위주로 진행하였습니다
  • 2단계에서는 프론트엔드와 소통하여 CORS 문제를 추가적으로 해결하고, 필요한 경우 HTTPS를 적용하여 PR을 제출하겠습니다!

yunjunghun0116 and others added 30 commits July 29, 2024 11:47
@yunjunghun0116
add: 주문하기 코드를 옮겨온다.
26aaca7 
옮겨온다.
@yunjunghun0116
refactor: gitignore 변경
63042e4
@yunjunghun0116
remove: 동시성 테스트 진행으로 인해 발생하는 build 과부하 방지를 위해 동시성테스트 제거
3496aae 
동시성 테스트 제거
@yunjunghun0116
add: 기존 동시성테스트 추가
8200094 
동시성 테스트 추가
@yunjunghun0116
add: ci-cd 파일 추가
624bc41 
ci-cd 파일 추가
@yunjunghun0116
rename: workflows -> workflow
0ea8930
@yunjunghun0116
rename: ci-cd to gradle
48074b0
@yunjunghun0116
Create gradle.yml
269eb4c
@yunjunghun0116
remove: workflow remove
ca96156
@yunjunghun0116
add: CI with java
998c03e
@yunjunghun0116
refactor: 자동으로 application.yml 파일 생성하도록 추가
ca2a6af
@yunjunghun0116
add: make application.yml 설정파일 생성
88ab80f
@yunjunghun0116
rename: application 설정파일 이름 재설정
28a71b5
@yunjunghun0116
remove: 동시성 테스트 삭제
711fd6c
@yunjunghun0116
change: setup jdk
e7400b4
@yunjunghun0116
remove: deploy 코드 제거
7ab6706
@yunjunghun0116
add: make application.yml 방식 추가
45a5f51
@yunjunghun0116
refactor: application.yml 만드는 코드 수정
d85e89a
@yunjunghun0116
add: deploy gradle 추가
6448271
@yunjunghun0116
rename: secret deploy server로 ip 보호
8e41a10
@yunjunghun0116
add: EC2 서버에 키를 알려줌
3af8063
@yunjunghun0116
add: 호스트키 신뢰하도록 설정
5a8031d
@yunjunghun0116
rename: 호스트키 이름 변경
42326ca
@yunjunghun0116
add: HOST 추가
a164ca9
@yunjunghun0116
add: 호스트키 검증 무시
0a52568
@yunjunghun0116
add: redirect-token-uri를 따로 관리해서 배포환경과 다른값 설정할 수 있도록 함
d3fb3f1
@yunjunghun0116
refactor: 이름 명시
5a33176
@yunjunghun0116
refactor: 배포 코드 수정
ed105f3
@yunjunghun0116
refactor: 절대 경로 설정
358c53a
@yunjunghun0116
add: 배포시 중간에 sleep 10 추가
2b9a9c5
@yunjunghun0116
refactor: 절대경로로 설정
8fc9d47
@yunjunghun0116
refactor: application.yml이 아닌 application.properties에 추가작성함으로써 하나로 통일관리
684570b
@yunjunghun0116
refactor: 설정값 변경
36429c5
@yunjunghun0116
add: 동시성테스트의 크기를 줄여 부하를 적게하여 업로드하기 쉽도록 변경
6f6daf4 
테스트코드 사이즈 변경
@yunjunghun0116
refactor: test부하 일부 증가
45a2c7d 
test부하 일부 증가
@yunjunghun0116
refactor: API 명세 팀원들과 통일
76e2444 
API 명세-URL을 팀원들과 통일
@yunjunghun0116
test: 수정된 API 명세에 알맞게 controller URL 변경
f2ef530
@yunjunghun0116
docs: Swagger 문서 작성을 위한 ApiResponse(Auth, Category) 작성 완료
142373f 
AuthApi, CategoryApi ApiResponse 정의 완료
@yunjunghun0116
docs: API 명세 통일에 따른 응답 반환
fdc7301 
응답 반환하기
@yunjunghun0116
remove: 중복되는 @hidden 삭제
831df81
@yunjunghun0116
infra: 스프링 배포 환경시 nohup 스프링 실행부분을 세션과 분리하여 종료
a34275d
@yunjunghun0116
infra: 백그라운드에서 실행 및 exit 으로 자동으로 배포까지하도록 설정
7c80fb7
@yunjunghun0116
infra: 백그라운드에서 스프링이 정상적으로 돌아가도록 gradle.yml 수정
d6bb361
@yunjunghun0116
add: 스프링실행시 세션을 정상적으로 종료할 수 있도록 한다
bd0d2d7
@yunjunghun0116
remove: Member-Role을 삭제
6c4f992
@yunjunghun0116
add: 예외 Response를 한번 더 감싸서 status와 함께 반환
bc1f139 
예외 Response 를 status, message 함께 반환
@yunjunghun0116
add: API 명세서 중 응답이 포함되지 않는 부분은 응답에서 제외한다.
5f8bc92 
Schema-hidden=true 설정을 통해 응답에서 제외했다.
@yunjunghun0116
refactor: 팀원들간 API 명세 통일 작업 수행
c971c7e 
API 명세 통일 작업 수행
asebn1
asebn1 approved these changes Aug 1, 2024
View reviewed changes
Copy link

@asebn1 asebn1 left a comment

Choose a reason for hiding this comment

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

정훈님 안녕하세요~
1단계 진행 잘해주셨네요 👍 👍
이번 단계는 이만 머지하겠습니다~
배포까지 마무리 잘해보아요!

@asebn1 asebn1 merged commit b72cae4 into kakao-tech-campus-2nd-step2:yunjunghun0116 Aug 1, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asebn1 asebn1 asebn1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yunjunghun0116 @asebn1