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

[#51] 이메일 인증 페이지 생성 #44

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

gyuZzang
Copy link
Member

@gyuZzang gyuZzang commented Apr 17, 2021

src/utils/api.js Outdated
export const API_SIGN_UP = makeAPI(POST, '/user');
export const API_FIND_ID = makeAPI(GET, '/user/id');
export const API_FIND_PW = makeAPI(GET, '/user/password');
export const API_SEND_CODE = makeAPI(POST, 'user/auth');
export const API_AUTH_EMAIL = (userID, code) => makeAPI(POST, `user/auth/${userID}/${code}`);
Copy link
Member

Choose a reason for hiding this comment

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

함수로 만든 건 의도하신 건가요??

Suggested change
export const API_AUTH_EMAIL = (userID, code) => makeAPI(POST, `user/auth/${userID}/${code}`);
export const API_AUTH_EMAIL = makeAPI(POST, `/user/auth`);

Copy link
Member Author

Choose a reason for hiding this comment

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

@Dormarble post (/user/auth) 에서 path params이 있는거랑 없는거랑 작동하는 방식이 다른데, api.js에는 하나로만 세팅하고, 프론트에서 API호출 시 setPathParams 세팅 여부에 따라 백에서 분기되는 건가요?

src/views/EmailAuth.jsx Outdated Show resolved Hide resolved
src/views/EmailAuth.jsx Outdated Show resolved Hide resolved
src/views/EmailAuth.jsx Outdated Show resolved Hide resolved
@gyuZzang gyuZzang self-assigned this Oct 4, 2021
@gyuZzang gyuZzang added feature New feature or improvement and removed In progress labels Oct 4, 2021
@Dormarble
Copy link
Member

API (POST api/user/auth/{authCode}) 수정 및 스웨거에 적용하였으니, 확인부탁드립니다
기존에 params에 userId 넘기는 부분 삭제

POST /api/user/auth/{userId}/{authCode}에서 다음과 같은 이유로 userId를 넣었던 것으로 예상합니다.

  1. 회원가입 시 이메일 인증 기능
    • 회원가입 시 이메일 인증 API 호출을 할때는 로그인을 하지 않아 토큰이 세팅되지 않을 것이기 때문에 서버에서 userId를 식별할 수 없음
  2. 확장성
    • 향후 추가 개발될 수 있는 인증 링크가 포함된 이메일 발송 방식에서는 URL만으로 유저를 식별할 수 있어야 함 (로그인이 되어 있든 말든 서버에서 유저를 식별할 수 있어야하기 때문)

해당 사항 고려하여 API 변경점 재검토 부탁드리겠습니다.

@gunhoflash
Copy link
Member

eslint 룰에 어긋나는 부분(quote, arrow-parens 등)이 많은데 확인이 필요합니다.

@gunhoflash
Copy link
Member

이메일 입력 없이 코드 전송 요청할 수 있어 보완이 필요해보입니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants