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

api 통일 부탁드립니다. #86

Open
HamBP opened this issue May 21, 2022 · 0 comments
Open

api 통일 부탁드립니다. #86

HamBP opened this issue May 21, 2022 · 0 comments

Comments

@HamBP
Copy link
Member

HamBP commented May 21, 2022

현재 로그인 성공시 jwt 혹은 sessionId 가 발급되어야 하는데 발급되지 않는 상태입니다. 따라서 로그인한 사용자에 대한 인증이 따로 없고 email 을 입력하는 방법으로 사용자 인증을 대체하고 있습니다.

그 과정에서 토큰이 들어가야할 자리에 아래와 같이 파라미터를 넘겨주어야 합니다.

image

image

image

하지만 어떤 곳에서는 email 을, 어떤 곳에서는 email + id 를, 어떤 곳에서는 seq 값을 요구하고 있습니다. 바로 jwt 나 sessionId 로 바꾸는 것이 어렵다면 파라미터라도 통일되어야 이후 리팩터링하기 수월할 것 같습니다.

또한 api 의 파라미터를 변경할 경우 많은 부분에서 클라이언트가 요청하는 많은 api 가 깨질 것으로 우려됩니다. 이럴 경우 기존 api 의 파라미터를 변경하는 것이 아닌 새로운 api 를 만들고, 기존 api 는 @deprecated 처리하는 것이 좋습니다.

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

No branches or pull requests

1 participant