We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
현재 로그인 성공시 jwt 혹은 sessionId 가 발급되어야 하는데 발급되지 않는 상태입니다. 따라서 로그인한 사용자에 대한 인증이 따로 없고 email 을 입력하는 방법으로 사용자 인증을 대체하고 있습니다.
그 과정에서 토큰이 들어가야할 자리에 아래와 같이 파라미터를 넘겨주어야 합니다.
하지만 어떤 곳에서는 email 을, 어떤 곳에서는 email + id 를, 어떤 곳에서는 seq 값을 요구하고 있습니다. 바로 jwt 나 sessionId 로 바꾸는 것이 어렵다면 파라미터라도 통일되어야 이후 리팩터링하기 수월할 것 같습니다.
또한 api 의 파라미터를 변경할 경우 많은 부분에서 클라이언트가 요청하는 많은 api 가 깨질 것으로 우려됩니다. 이럴 경우 기존 api 의 파라미터를 변경하는 것이 아닌 새로운 api 를 만들고, 기존 api 는 @deprecated 처리하는 것이 좋습니다.
The text was updated successfully, but these errors were encountered:
No branches or pull requests
현재 로그인 성공시 jwt 혹은 sessionId 가 발급되어야 하는데 발급되지 않는 상태입니다. 따라서 로그인한 사용자에 대한 인증이 따로 없고 email 을 입력하는 방법으로 사용자 인증을 대체하고 있습니다.
그 과정에서 토큰이 들어가야할 자리에 아래와 같이 파라미터를 넘겨주어야 합니다.
하지만 어떤 곳에서는 email 을, 어떤 곳에서는 email + id 를, 어떤 곳에서는 seq 값을 요구하고 있습니다. 바로 jwt 나 sessionId 로 바꾸는 것이 어렵다면 파라미터라도 통일되어야 이후 리팩터링하기 수월할 것 같습니다.
또한 api 의 파라미터를 변경할 경우 많은 부분에서 클라이언트가 요청하는 많은 api 가 깨질 것으로 우려됩니다. 이럴 경우 기존 api 의 파라미터를 변경하는 것이 아닌 새로운 api 를 만들고, 기존 api 는 @deprecated 처리하는 것이 좋습니다.
The text was updated successfully, but these errors were encountered: