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

refactor : 회원가입시 닉네임 중복을 할 수 없게함 #275

Closed
1 task
s-hwan opened this issue Jan 8, 2025 · 5 comments · Fixed by #276
Closed
1 task

refactor : 회원가입시 닉네임 중복을 할 수 없게함 #275

s-hwan opened this issue Jan 8, 2025 · 5 comments · Fixed by #276
Assignees
Labels
refactoring 리팩토링

Comments

@s-hwan
Copy link
Contributor

s-hwan commented Jan 8, 2025

📌 Issue

-회원가입시 닉네임 , 이메일 등 중복 방지를 하겠습니다

✅ TODO

  • singup API 수정
@s-hwan s-hwan added the refactoring 리팩토링 label Jan 8, 2025
@s-hwan s-hwan self-assigned this Jan 8, 2025
@rladmstn
Copy link
Contributor

rladmstn commented Jan 8, 2025

닉네임 중복 외에도 이메일 중복이나 유효성 등 검사할 수 있는 것들은 2차 검사 모두 진행해주면 좋을 것 같아요!

@s-hwan
Copy link
Contributor Author

s-hwan commented Jan 8, 2025

닉네임 중복 외에도 이메일 중복이나 유효성 등 검사할 수 있는 것들은 2차 검사 모두 진행해주면 좋을 것 같아요!

네 알겠습니다!
질문이 있는데 프론트에서는 중복된 닉네임이라고 안되는데 서버에 api 보낼 때는 되니까 두 번 막는다고 생각하면 되는건가요?

@rladmstn
Copy link
Contributor

rladmstn commented Jan 8, 2025

네 알겠습니다! 질문이 있는데 프론트에서는 중복된 닉네임이라고 안되는데 서버에 api 보낼 때는 되니까 두 번 막는다고 생각하면 되는건가요?

넵 맞습니다-!

@hwangjokim
Copy link
Contributor

네맞아요

서버 개발자에게 두 개 필수 원칙이 있습니다.

  1. 클라이언트를 믿지 마라
  2. 조건 1을 절대 잊지 마라

그냥 유저말고 악용하려고 눈에 불킨 사람만 있다고 생각해주세요
악용말고도, 유저 두 명이 동시에 클라이언트에서 같은 닉네임을 검증하고, 회원가입을 누르면 클라검증을 통과하겠죠~

@s-hwan
Copy link
Contributor Author

s-hwan commented Jan 8, 2025

네맞아요

서버 개발자에게 두 개 필수 원칙이 있습니다.

  1. 클라이언트를 믿지 마라
  2. 조건 1을 절대 잊지 마라

그냥 유저말고 악용하려고 눈에 불킨 사람만 있다고 생각해주세요 악용말고도, 유저 두 명이 동시에 클라이언트에서 같은 닉네임을 검증하고, 회원가입을 누르면 클라검증을 통과하겠죠~

오 그런 경우도 생각할 수 있군요..!

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

Successfully merging a pull request may close this issue.

3 participants