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-6] DEV용 access token을 받아오는 api 작성 #101

Merged
merged 3 commits into from
Mar 18, 2024

Conversation

05AM
Copy link
Collaborator

@05AM 05AM commented Mar 16, 2024

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

  • access token을 얻기 위해 매번 로그인을 하는 것이 번거롭습니다.

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

  • dev용 access token 발급 api를 만들었습니다.

@05AM 05AM added the D-1 1일 전 까지 리뷰해주세요 label Mar 16, 2024
@05AM 05AM self-assigned this Mar 16, 2024
@05AM 05AM requested a review from zxcv9203 as a code owner March 16, 2024 18:53
@05AM 05AM changed the title [STMT-6] DEV용 access token을 받아오는 api 작성 ✨[STMT-6] DEV용 access token을 받아오는 api 작성 Mar 16, 2024
Copy link
Collaborator

@zxcv9203 zxcv9203 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

나중에 보안을 위해 다음 사항을 고려하면 더 좋을 것 같습니다.

  1. 특정 권한을 가진 유저만 AccessToken을 발급 받을 수 있도록 처리(새로운 Role을 추가하여 TEST 권한인 유저만 해당 API로 생성이 가능하다던지.. 아니면 테스트용 계정만 AccessToken을 발급 받을 수 있도록 처리한다던지..)
  2. 특정 프로파일(dev, local)에서만 해당 API를 호출할 수 있도록 처리

@github-actions github-actions bot added D-0 바로 리뷰가 필요해요 and removed D-1 1일 전 까지 리뷰해주세요 labels Mar 17, 2024
@05AM
Copy link
Collaborator Author

05AM commented Mar 18, 2024

나중에 보안을 위해 다음 사항을 고려하면 더 좋을 것 같습니다.

제안해주신 방법들 모두 좋은 방법이라고 생각합니다! 개인적으로 1번 방법이 해당 test 유저 관련된 정보만 변경할 수 있게 되어 좋은 것 같습니다.
추후에 반영하여 올리겠습니다!

@05AM 05AM merged commit 08933e3 into dev Mar 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-0 바로 리뷰가 필요해요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants