Skip to content

[fix] #25 - 회원 로직 및 배포 프로세스 픽스#26

Merged
jjinleee merged 13 commits intomainfrom
fix/#25-회원로직버그
Apr 25, 2025

Hidden character warning

The head ref may contain hidden characters: "fix/#25-\ud68c\uc6d0\ub85c\uc9c1\ubc84\uadf8"
Merged

[fix] #25 - 회원 로직 및 배포 프로세스 픽스#26
jjinleee merged 13 commits intomainfrom
fix/#25-회원로직버그

Conversation

@jjinleee
Copy link
Member

@jjinleee jjinleee commented Apr 25, 2025

1. 회원로직 재검

1️⃣ redirect URI 인코딩 수정

2️⃣ savedCafe 선삭제 후 탈퇴

3️⃣ 카톡로그인 email null값으로 들어옴→동의 안되어있어서 그럼
→ scope에 빠져있어서 Dockerfile에 추가
수정코드
Dspring.security.oauth2.client.registration.kakao.scope=profile_nickname,profile_image,account_email \

4️⃣ Logout Redirect URI 카카오 디벨로퍼에 추가 완료



2. EC2 부담 감소위한 deploy.yml 수정방안 적용완료

✅ 1.빌드를 로컬(GitHub Actions)에서 하고, EC2에는 결과물만 전달
✅ 2. jar 파일 경로 문제 수정
✅ 3. jar 파일 EC2로 전송
✅ 4. Dockerfile 수정

항목 기존 방식 개선 방식
빌드 위치 EC2 내부에서 ./gradlew build GitHub Actions에서 빌드 후 EC2로 복사
부하 발생 위치 EC2 (CPU 부담) GitHub Actions (EC2는 실행만)
Docker build EC2에서 직접 수행 GitHub Actions에서 수행 후 docker save


✔️ 바뀌는 흐름

  1. GitHub Actions에서 JAR 빌드
  2. 빌드된 JAR 파일만 EC2로 전송
  3. EC2는 docker-compose up만 수행 (--build 없음)
  4. Dockerfile은 이미 JAR 경로를 COPY하는 구조여야 함

@jjinleee jjinleee linked an issue Apr 25, 2025 that may be closed by this pull request
2 tasks
@jjinleee jjinleee self-assigned this Apr 25, 2025
@jjinleee jjinleee requested a review from ghi512 April 25, 2025 04:31
@jjinleee jjinleee added the Fix label Apr 25, 2025
Copy link
Member

@ghi512 ghi512 left a comment

Choose a reason for hiding this comment

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

탈퇴 잘되네용 !!! 저장카페 없어지고, 리뷰랑 제보는 남아있고! 고생하셨슴다
굳굳굳 ~~~

배포로직 바꾼 것도 짱 ~ 이제 ec2 안죽었으면 조켓네요 ..

image

머지할 때 deploy.yml에 요 브랜치는 삭제하시나용?? 아님 상관없나요?!

@jjinleee
Copy link
Member Author

상관없긴 한데 삭제하려구요~

@jjinleee jjinleee merged commit 5929d9c into main Apr 25, 2025
2 checks passed
@jjinleee jjinleee deleted the fix/#25-회원로직버그 branch April 25, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 회원로직 버그

2 participants