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

[chore] next.js CI Github Action 완성 #177

Merged
merged 23 commits into from
Aug 13, 2024
Merged

[chore] next.js CI Github Action 완성 #177

merged 23 commits into from
Aug 13, 2024

Conversation

MinhoJJang
Copy link
Member

@MinhoJJang MinhoJJang commented Aug 8, 2024

📑 구현 사항

main 브랜치에 pull_request 를 통해 머지될 경우 깃액션이 돕니다.
yml과 Dockerfile을 npm sharp 을 통해 최적화하였습니다.
각 배포 시 마다, Release 버전이 자동으로 업데이트 됩니다.
KCR까지 이미지가 정상적으로 업로드되면 성공입니다.


🌁 결과 화면

과정 살펴보기


연관 이슈


코멘트

  • 없습니다.

@MinhoJJang MinhoJJang requested a review from Ginieee August 8, 2024 06:59
@MinhoJJang MinhoJJang self-assigned this Aug 8, 2024
Copy link
Collaborator

@Ginieee Ginieee left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 이거 머지해도 되는 건가요 ??

@MinhoJJang
Copy link
Member Author

수고하셨습니다! 이거 머지해도 되는 건가요 ??

결론부터 말씀드리면, 아직 안정적인 상태는 아닙니다. 제가 해당 PR을 DRAFT로 변경하려고 ORGANIZATION 권한을 요청드렸는데, 가능하시다면 권한을 부여해 주시면 감사하겠습니다.

현재 사이트 접속 시 알 수 없는 오류가 발생하여 조사 중입니다만, 제가 Next.js에 대한 경험이 부족하여 아직 원인을 파악하지 못하고 있습니다. 정상적으로 로그인 및 백엔드 통신을 확인한 후에 병합을 진행하고자 합니다.

  1. 혹시 어진님께서 https://dkation.com/login 사이트에 접속하셨을 때, 콘솔에서 어떤 에러가 발생하는지 알려주실 수 있으실까요?
    저는 아래와 같은 오류를 확인하고 있습니다:
    image

  2. 로그인 시 어떤 파일에서 어떠한 로직으로 작동하는지, 그리고 그 과정에서 사용되는 URL, DB 접근, 환경변수 등에 대해 설명해 주실 수 있으시다면 매우 감사하겠습니다. 현재 문제가 발생하는 부분을 파악하는 데 큰 도움이 될 것 같습니다.

@Ginieee
Copy link
Collaborator

Ginieee commented Aug 11, 2024

수고하셨습니다! 이거 머지해도 되는 건가요 ??

결론부터 말씀드리면, 아직 안정적인 상태는 아닙니다. 제가 해당 PR을 DRAFT로 변경하려고 ORGANIZATION 권한을 요청드렸는데, 가능하시다면 권한을 부여해 주시면 감사하겠습니다.

현재 사이트 접속 시 알 수 없는 오류가 발생하여 조사 중입니다만, 제가 Next.js에 대한 경험이 부족하여 아직 원인을 파악하지 못하고 있습니다. 정상적으로 로그인 및 백엔드 통신을 확인한 후에 병합을 진행하고자 합니다.

  1. 혹시 어진님께서 https://dkation.com/login 사이트에 접속하셨을 때, 콘솔에서 어떤 에러가 발생하는지 알려주실 수 있으실까요?
    저는 아래와 같은 오류를 확인하고 있습니다:
    image
  2. 로그인 시 어떤 파일에서 어떠한 로직으로 작동하는지, 그리고 그 과정에서 사용되는 URL, DB 접근, 환경변수 등에 대해 설명해 주실 수 있으시다면 매우 감사하겠습니다. 현재 문제가 발생하는 부분을 파악하는 데 큰 도움이 될 것 같습니다.

본 레포지토리에서는 이미 admin으로 되어 있으셔서, 어느 부분에서 권한을 드려야하는지 잘 모르겠네요.. ! 그냥 PR 말머리를 [draft] 로 바꿔주셔도 괜찮을 것 같습니다! 말씀하신 이슈에 관해서는 저도 동일한 이슈가 뜹니다. 저도 처음 보는 에러라 좀 더 찾아봐야 할 것 같습니다.

로그인과 관련해서는 src/app/_components/user/login/UserLoginSection.tsx 파일에서, handleLogin 함수 내에 있는

    const result = await signIn('credentials', {
      accountId: form.accountId,
      password: form.password,
      redirect: false,
    });

부분에서 실행하고 있습니다. NextAuth의 signIn 함수를 사용하고 있어서, src/app/api/auth/[...nextauth]/route.ts 이 부분에 있는 authorize 내부의 저희 백엔드 쪽 로그인 통신이 진행된다고 보시면 됩니다. import한 api 는 저희 백엔드 서버의 기본 URL을 담고 있으며, 이러한 처리를 거친 후 리다이렉트(또는 NextAuth의 권한 관리 영역을 따지는) URL 로 NEXTAUTH_URL이 사용된다고 보시면 될 것 같습니다.

저도 해당 이슈 방법을 찾아보겠으나, 계속 이슈가 유지된다면 멘토링 요청도 필요할 것 같습니다! 고려해보시고 진행 부탁드립니다. 이슈 관련해서 알아낸 게 있으면 연락드리겠습니다!

@MinhoJJang MinhoJJang changed the title [chore] next.js CI Github Action 완성 [draft] [chore] next.js CI Github Action 완성 Aug 11, 2024
@MinhoJJang MinhoJJang changed the title [draft] [chore] next.js CI Github Action 완성 [chore] next.js CI Github Action 완성 Aug 13, 2024
@MinhoJJang MinhoJJang requested a review from Ginieee August 13, 2024 08:23
@MinhoJJang
Copy link
Member Author

MinhoJJang commented Aug 13, 2024

관련 에러 모두 해결하였고 정상적으로 로그인 되는 것 확인하였습니다!

또한 현재 dev 브랜치를 main 으로 머지해도 괜찮을까요?

@Ginieee
Copy link
Collaborator

Ginieee commented Aug 13, 2024

관련 에러 모두 해결하였고 정상적으로 로그인 되는 것 확인하였습니다!

또한 현재 dev 브랜치를 main 으로 머지해도 괜찮을까요?

노션 확인했습니다! 고생 많으셨군요... 🥲 해결돼서 다행이에요! 머지 진행하면 될 것 같아서, 본 PR 머지하도록 하겠습니다! dev 브랜치 머지는 오늘 밤(~새벽) 작업 후에 진행하고 싶은데 괜찮을까요??

@MinhoJJang
Copy link
Member Author

관련 에러 모두 해결하였고 정상적으로 로그인 되는 것 확인하였습니다!

또한 현재 dev 브랜치를 main 으로 머지해도 괜찮을까요?

노션 확인했습니다! 고생 많으셨군요... 🥲 해결돼서 다행이에요! 머지 진행하면 될 것 같아서, 본 PR 머지하도록 하겠습니다! dev 브랜치 머지는 오늘 밤(~새벽) 작업 후에 진행하고 싶은데 괜찮을까요??

넵 편하실때 해주시면 됩니다 항상 수고 많으십니다 ㅎㅎ

@Ginieee Ginieee merged commit d865dc9 into main Aug 13, 2024
1 check passed
Copy link

@MinhoJJang MinhoJJang deleted the MF-255-Next.js-WEB-CI branch August 14, 2024 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants