2023-08-16 정기 개발 회의

[Jake1152]

2023-08-09 정기 개발 회의

[Jake1152]

집현전 기술블로그

• DTO맞추는 리팩터링 과정을 글로 표현하면 좋은 경험, 나중에 새로운 개발팀 들어오시는 분들에게 내용전달 그리고 취업에 도움될 거 같습니다

[jimin52]

모두의 포트폴리오

[scarf005]

announcement 탭을 활용하면 큰 준비 없이 간편하게 시작 가능할 것 같아용

[Jake1152]

1달 뒤쯤 바쁜거 지나고 진행

[not-using]

현재 스웨거 확인할 수 있는 경로가 바뀌었을까요?
https://server.42library.kr/swagger/ 접속이 안됩니다

[scarf005]

jiphyeonjeon-42/backend#640
버그픽스는 develop에 머지되었으나 아직 배포가 되지 않아 발생한 문제인것 같습니다.
cc @jhMin95

[not-using]

https://dev.42library.kr/swagger/로 확인하기로

[Jake1152]

• https만 제공되는 중
• cookie 안들어가는 이슈
• jimin님의 webserv 경험 했을 때
• 헤더 설정 안해서 였다
• header에 credentail true이어야 바뀐다.

[not-using]

https://github.com/jiphyeonjeon-42/frontend/issues/427

/auth/logout 호출을 보내지만 access_token을 없애지 않습니다
HTTP only 쿠키라서 백에서 지워주세요

• 이슈 Transfer

[scarf005]

jiphyeonjeon-42/frontend#537 (comment)

프론트에 동기화 문제가 있는 것 같습니다.

[not-using]

프론트 문제 아닙니다

localstorage의 값은 /auth/me로 받아 전역상태로 저장해서 UI에 뿌리기 위한 데이터로 access_token과는 무관해요
특히 expire값은 JWT시크릿을 쓰지 않고도 유효기간을 임의로 계산하기 위해서 넣은 것일 뿐 실제 토큰에 영향을 주지 않습니다
그냥 로그아웃을 오래 하지 않은 유저(분명 토큰이 만료되었을 유저)에게 혼동을 주지 않기 위해 넣은 편의기능이라 생각해주심 좋을것 같습니다
그리고 로그아웃 시도시 로컬스토리지의 내용은 제대로 삭제되고 있습니다!!

이슈 본문에서 언급했듯 쿠키에 남아있는 access_token의 문제이며 HTTP only쿠키라서 백엔드에서 지워주셔야 해요

[scarf005]

프론트에 동기화 문제가 있는 것 같습니다.

조금 더 상세히 설명해야했는데, 로그인 요청이 실패했을때 로컬스토리지의 user 키가 삭제되지 않는 점에 대한 이야기였습니다.

[not-using]

언급되고 있는 이슈는 로그아웃시 토큰이 사라지지 않는 문제라 다른 문제인것 같아요

새로운 버그는 새 이슈로 만들어주세요
HTTP 401 에러일 때 스토리지를 삭제하는 부분을 처리해줘야 할 것 같습니다!

[scarf005]

부탁드릴게요!

[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)

intra_oauth 테이블 create

CREATE TABLE `intra_oauth` (
`id` INT NOT NULL AUTO_INCREMENT PRIMARY KEY,
`userId` INT NOT NULL,
`slack` VARCHAR(255) UNIQUE,
`intraId` INT UNIQUE,
`isValid` BOOLEAN DEFAULT FALSE,
`createdAt` DATETIME(6) NOT NULL DEFAULT CURRENT_TIMESTAMP(6),
`updatedAt` DATETIME(6) NOT NULL DEFAULT CURRENT_TIMESTAMP(6) ON UPDATE CURRENT_TIMESTAMP(6),
INDEX (`userId`) );

intra_oauth 테이블에 값 입력

INSERT INTO intra_oauth (userId, intraId, slack, createdAt, updatedAt) 
SELECT id, intraId, slack, createdAt, updatedAt FROM user;

users 테이블에 값 입력

INSERT INTO users (id, nickname, penaltyEndDate, role, createdAt, updatedAt) 
SELECT id, nickname, penaltyEndDate, role, createdAt, updatedAt FROM user;

google oauth api 생성

참고: https://www.joinc.co.kr/w/man/12/oAuth2/Google
일단 생성은 해뒀습니다.
redirection, url 설정 등등 추가적으로 설정 필요

요구사항 분석

1. 회원가입이 Intra 로 들어왔다 -> users 계정 만들기
2. 회원가입이 google 로 들어왔다 -> users 계정 만들기
3. google 유저는 Intra 계정인증 해야 한다. staff 인증은 어떤 식으로 할 것인지 정해야 할 듯.
4. google 유저가 Intra 계정 연동하기
   1. 기존 Intra-oauth 계정이 있음 -> google-oauth 인증된 users 계정을 삭제하고 기존 Intra 인증된 users 계정에 통합
   2. 기존 Intra-oauth 계정이 없음 -> users 계정으로 새로운 intra-oauth 생성. role 변경

v2 auth api 명세

작성중...

생각해야 하는 일

• 닉네임은 어떻게 설정할지?
• 결국 intra 로그인이 메인이 될 수 밖에 없는 구조
• access token, refresh token 적용
• email, slackId 등 개인정보를 DB 에 저장 할 지
• google access token 으로 뭐하지??

[scarf005]

기존 Intra-oauth 계정이 있음 -> google-oauth 인증된 users 계정을 삭제하고 기존 Intra 인증된 users 계정에 통합

혹시 users 계정을 삭제해야 하는 이유에 대해 설명해주실수 있을까요? 이런 식으로 진행될 수 있지 않을까 하는 생각이 들었는데, 혹시 이 방식대로 하면 생길 수 있는 문제점이 있을지 궁금합니다.

google에 연동된 유저가 42 연동

google에 연동된 상태

• user 테이블 (userId)
  • google 테이블 (googleId)

42 연동 후

• user 테이블 (userId)
  • google 테이블 (googleId)
  • 42 테이블 (42Id)

서버에서는 userId를 알고 있으므로, 42 테이블을 생성하고 user 테이블의 userId와 연결

42에 연동된 유저가 google 연동

42에 연동된 상태

• user 테이블 (userId)
  • 42 테이블 (42Id)

google 연동 후

• user 테이블 (userId)
  • 42 테이블 (42Id)
  • google 테이블 (googleId)

서버에서는 userId를 알고 있으므로, google 테이블을 생성하고 user 테이블의 userId와 연결

[scarf005]

개발 서버 진행 공유

경로: https://dev.42library.kr

lightsail 인스턴스에서 정상적으로 동작하나 쿠키 관련 문제가 발생해 잠시 정지시킨후 수정중입니다.

원인

기존 환경 (로컬)

백엔드	프론트엔드
http://localhost:3000	http://localhost:4242

-> 도메인 일치, 포트만 다름

기존 환경 (배포)

백엔드	프론트엔드
https://server.42library.kr	https://42library.kr

-> 도메인(42library.kr)과 서브도메인이 일치

개발 서버

백엔드	프론트엔드
https://dev.42library.kr	https://localhost:4242

기존 환경과는 달리 도메인이 다르기 때문에 CORS를 적용해야 하는데, 필요한 플래그를 넣어주었음에도 Set-Cookie 헤더가 오류 메시지 없이 적용이 되지 않고 있어 조사중입니다.

[Jiwon-Woo]

개발서버 프론트엔드 http가 아니라 https 맞나요?

[scarf005]

네 https으로 접속해주시면 되요

[not-using]

42 api 리다이렉트 문제 : 키를 새로 생성함 (youkim님 계정에 연동됨)