2023-10-25 정기 개발 회의

[jimin52]

회의회의

[Jake1152]

백앤드 10-23 회의 내용 공유

6차 개발 사항 논의, 개발 사항 분담

• v2에 리팩터링 진행

  • Auth api 작성 및 DB이관
  • typeORM를 Kysely 쿼리 빌더로 변경
  • Reviews 테스트 로직 추가
  • 타입안정성 높이기
  • front에서도 따로 정의 필요없이 backend에서 정의한 것을 가져다 쓰는 식으로 변경
  • typescript type 사용
  • contract를 패키지로 만들어서 사용 가능

• 상세 내용 아래와 같음

  • 2023-10-23 백엔드 정기 회의 backend#787

[nyj001012]

진행 상황 (V2)

• histories API에 kysely 적용
• node test runner를 reviews API에 시험 사용

[Jake1152]

• Q1, api의 완성 여부는 어떻게 알 수 있는가?
  • swagger에 있으면 구현된 endpoint이다
    • PR에서 swagger항목이 있기에 swagger에 있으면 확실하게 구현된 것임 알 수 있다

[jimin52]

백엔드 소셜로그인 진행상황 공유

DB 설계

### user
| column         | type         | Null | Key | Default             | Extra                                          |
| -------------- | ------------ | ------ | -------- | -------------------- | ------------------------------------------------ |
| id             | int          | NO     | PRI      | NULL                 | auto_increment                                   |
| email          | varchar(255) | NO     | UNI      | NULL                 |                                                  |
| password       | varchar(255) | NO     |          | NULL                 |                                                  |
| nickname       | varchar(255) | YES    |          | NULL                 |                                                  |
| intraId        | int          | YES    | UNI      | NULL                 |                                                  |
| slack          | varchar(255) | YES    | UNI      | NULL                 |                                                  |
| penaltyEndDate | datetime     | NO     |          | CURRENT_TIMESTAMP    | DEFAULT_GENERATED                                |
| role           | tinyint      | NO     |          | 0                    |                                                  |
| createdAt      | datetime(6)  | NO     |          | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED                                |
| updatedAt      | datetime(6)  | NO     |          | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED on update CURRENT_TIMESTAMP(6) |

### users
| column         | type         | Null | Key | Default           | Extra             |
| -------------- | ------------ | ---- | --- | ----------------- | ----------------- |
| id             | int          | NO   | PRI | NULL              | auto_increment    |
| nickname       | varchar(255) | YES  |     | NULL              |                   |
| penaltyEndDate | datetime     | NO   |     | CURRENT_TIMESTAMP | DEFAULT_GENERATED |
| role           | tinyint      | NO   |     | 0                 |                   |
| isActive       | boolean      | NO   |     | FALSE             |                   |
| createdAt      | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED                                |
| updatedAt      | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED on update CURRENT_TIMESTAMP(6) |

### google_oauth
| column               | type         | Null | Key | Default              | Extra                                            |
| -------------------- | ------------ | ---- | --- | -------------------- | ------------------------------------------------ |
| id                   | int          | NO   | PRI | NULL                 | auto_increment                                   |
| userId               | int          | NO   | MUL | NULL                 |                                                  |
| email                | varchar(255) | NO   | UNI | NULL                 |                                                  |
| google_access_token  |              |      |     |                      |                                                  |
| google_refresh_token |              |      |     |                      |                                                  |
| createdAt            | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED                                |
| updatedAt            | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED on update CURRENT_TIMESTAMP(6) |

### intra_oauth
| column              | type         | Null | Key | Default              | Extra                                            |
| ------------------- | ------------ | ---- | --- | -------------------- | ------------------------------------------------ |
| id                  | int          | NO   | PRI | NULL                 | auto_increment                                   |
| userId              | int          | NO   | MUL | NULL                 |                                                  |
| slack               | varchar(255) | YES  | UNI | NULL                 |                                                  |
| intraId             | int          | YES  | UNI | NULL                 |                                                  |
| intra_access_token  |              |      |     |                      |                                                  |
| intra_refresh_token |              |      |     |                      |                                                  |
| createdAt           | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED                                |
| updatedAt           | datetime(6)  | NO   |     | CURRENT_TIMESTAMP(6) | DEFAULT_GENERATED on update CURRENT_TIMESTAMP(6) |

google oauth 라이브러리 발견

api 엔드포인트 설계

POST `/api/v2/auth/login/intra`
intra 로 이동해서 인증

POST `/api/v2/auth/login/google`
google 로 이동해서 인증

POST `/api/v2/auth/token/intra`
intra 토큰 재발급

POST `/api/v2/auth/token/google`
google 토큰 재발급

GET `/api/v2/auth/me`
로그인 정보 얻어오기

POST `/api/v2/auth/logout`
로그아웃. 토큰 초기화

POST `/api/v2/auth/register/intra`
인트라 회원가입

POST `/api/v2/auth/register/google`
구글 회원가입

[Jake1152]

• Q4. 꽤 걸릴 것 같다
  • 구현자체는 오래 걸리지 않을 것 같다
  • 배포에서의 난이도가 있을 거 같다.
  • 하는 김에 v2 user도 배포

[Jake1152]

• Q5 . 집현전 회원가입을 [email protected]으로 가입한 이후에 [email protected]으로 바꾸고 싶다면 가능한가?

  • 별도 처리가 필요

• Q6. 토큰 재발급 post를 보내는 상황은 무엇인가?

  • 다른 api 요청이 401일때의 상황이다
  • 만료된 토큰인지 아닌지를 error status code로 줄 수 있을거 같다. 이를 통해 에러 메시지를 보내 줄 수 있다
  • Q6-1 error code는 어떻게 되는가?
    • http error code로 전송됨

[jimin52]

보완점:

• userRepository 참조하던 기존 api 수정 필요할 듯
• user 테이블 관련 이슈를 보완해야 할 듯 합니다

[Jake1152]

• 주의 사항
  • 서버에서 노드 빌드하다가 터지는 경우가 있다고 한다 그러므로 베포단에서 신경 쓸 것이 많다

[Jake1152]

• env 자동 배포가 급하다
• git actiond으로 env만 변경해서 배포하면 좋을 거 같다. CI/CD 관심있는 사람(인셉션 이상)

[Jake1152]

프론트앤드

• 월요일 코어 타임마다 만나서 집현전 이슈 확인 및 해결

[chanhihi]

프론트엔드 10-23 회의 내용 공유

jiphyeonjeon-42/frontend#579

• 월요일 코어 타임마다 만나서 집현전 이슈 확인 및 해결

[not-using]

aws -> oci 이전

11월 4일 진행

[Jake1152]

로그인 이슈 해결 되었는가?

• 어느순간 로그인이 풀렸는데 my page로 들어가면 1번 유저로 로그인 되는 이슈

[Jake1152]

• 내부적으로는 같은 원인으로 파악
  • typeORM undefined 인자가 들어오면 table의 첫번째 row를 return하는 문제가 있다
  • 백앤드 로직에서 type check가 필요하다