SWM14-Architect / moview-core-service

모뷰의 백엔드 API 서버
5 stars 0 forks source link

Feat: Kakao OAuth 추가 #103

Closed westreed closed 1 year ago

westreed commented 1 year ago

Kakao OAuth 추가

  1. @dasd412 님이 작업.
  2. Access Token과 Refresh Token까지 적용됨.
  3. Access Token은 만료시간이 30분(1800), Refresh Token은 1주일(604800)로 설정되어 있음.

EnviromentLoader 적용

  1. 기존 코드에서 환경변수를 Local에서 가져오도록 되어있어서, EnviromentLoader를 적용.
  2. 현재 Kakao OAuth 계정은 @westreed 껄로 되어있어서 모뷰용 카카오계정을 생성후 AWS 환경변수에 반영해야함.

OAuth 적용에 따른 Session 방식에서 JWT로 교체

  1. Session id를 쓰던 방식을 전부 제거하고 OAuth에서 받은 jwt_identity로 대체함. (이부분 맞게 한건지 검토바람..)
  2. 네이밍 반영 : InterviewSession을 Interview로 변경.

기존에 있던 오류 수정

  1. interview Document의 input data document id가 비어있던 오류 수정함.
  2. interview의 previous_question_content에는 문서가 생성되면서 초기질문도 함께 들어가는데, Answer API에서도 질문을 추가하고 있어서 초기질문이 중복으로 들어가는 문제가 발생하여 수정함.
westreed commented 1 year ago

수고하셨습니다. 현재 Kakao OAuth 계정은 @westreed 껄로 되어있어서 모뷰용 카카오계정을 생성후 AWS 환경변수에 반영해야함. 이 부분은 environment_loader.py의 USER_SPECIFIC_PARAMETERS에 kakao-client-id와 같은 값을 추가해놓으면 각자 개발 환경에서는 각자 아이디를 이용할 수 있도록 할 수 있습니다. 실제로 모뷰용 카카오계정을 만들더라도, 개발을 할 때에는 모뷰 계정보다는 각자의 아이디를 이용하는 게 편리할 것 같아서 위 방법을 추천드립니다.

넵 거기에 추가해서 각자 쓰게할까 했었는데 모뷰계정을 생성할거면 굳이 따로 쓸 필요는 없을 것 같더라구요... 라고 답변하려고 했는데 생각해보니 Kakao OAuth 설정에서 도메인 설정도 필요하고 1개만 지정이 가능해서 말씀하신대로 하는게 맞을 듯 합니다.