Skip to content

[Fix] 로그인 관련 이슈 해결#183

Open
COrollaROOT wants to merge 4 commits intodevelopfrom
Feat-KYG/GPGS2
Open

[Fix] 로그인 관련 이슈 해결#183
COrollaROOT wants to merge 4 commits intodevelopfrom
Feat-KYG/GPGS2

Conversation

@COrollaROOT
Copy link
Collaborator

🧑‍💻 PR

🔗 관련 이슈

  • 수행한 작업과 관련 작업 번호를 작성해주세요

#162
#142

이슈 관련 버구 수정

닉네임 검사 통과 문제(자음만/모음만 허용) 수정

🔥 작업 내용 요약

Feat – 닉네임 유효성 규칙 추가: “자음만” 또는 “모음만”으로만 구성된 닉네임 금지

Feat – NicknameRules.cs 유틸 신규 추가(한글 자모/음절 판별 포함)

Fix – 닉네임 입력 UI(NicknamePopup 등)에서 확인 버튼 클릭 시 신규 규칙 적용

정규식 상수에 “자음만/모음만” 패턴 추가
자음(ㄱㅎ 등): \u3131-\u314E(Compatibility Jamo), \u1100-\u115F(Jamo L), \uA960-\uA97F(Jamo Ext-A 일부)
모음(ㅏ
ㅣ 등): \u314F-\u3163(Compatibility Jamo V), \u1160-\u11A7(Jamo V)


💻 작업 구현 방법 (공통)

📝 구현 설명

  • 어떤 방식으로 구현했는지, 주요 로직과 처리 방식 설명

닉네임의 모든 문자를 스캔해서 “한글 자모(자/모)”만으로 구성되었는지 판단합니다.
전부 자음 자모만이거나, 전부 모음 자모만이면 거절합니다.

한글 완성형 음절(가–힣), 영문/숫자/기타 문자가 하나라도 섞이면 이 규칙엔 걸리지 않습니다(즉, 통과).
요구사항이 “자음만” 또는 “모음만” 금지이므로 최소 수정 범위로 반영했습니다.

한글 호환 자모(Compatibility Jamo): U+3131–U+314E(자음), U+314F–U+3163(모음)
한글 자모(Jamo): 자음 U+1100–U+115F, 종성자음 U+11A8–U+11FF, 모음 U+1160–U+11A7
한글 음절(Syllables): U+AC00–U+D7A3 → 음절이 끼면 “자/모만” 조건이 깨지므로 통과

❓구현 의도

  • 해당 방식으로 구현한 이유, 선택한 설계 방향 공유

기존 필터는 “자음만/모음만” 닉네임을 구별하지 못해 읽기 불가/의미 불명 닉네임이 통과했습니다.
최소한의 정책 변경으로 부작용을 줄이기 위해 “전부 자음만” 또는 “전부 모음만”인 케이스만 차단합니다.
필요 시 추후 옵션으로 “자모가 섞여만 있어도 금지(엄격 모드)”로 확장 가능하도록 함수 구조를 분리

🔊 특이사항

  • 주의해야 할 점 (중요 변경 사항 요약)

💻 버그 해결방법 (Fix 전용)

🔧 수정된 버그

닉네임이 자음만(예: ㄱㄴㄷ, ㅎㅎㅎ) 또는 모음만(예: ㅏㅣㅗ)일 때도 통과하던 문제

💡 해결방법

  • 어떠한 방식을 해결을 했는지 설명

NicknameRules.IsOnlyConsonantJamo() / IsOnlyVowelJamo() 검사로 전부 자음/전부 모음 케이스를 판별하고 거절

✔️ 버그 체크리스트

  • 추가로 수정이 필요함
  • 수정 완료
  • QA 테스트 통과

💬 기타 사항

  • 수정이 안된 경우 필요한 작업이 뭔지 진행 상황 공유

✅ PR 체크리스트

  • 빌드 오류 없음
  • 기능 정상 작동 확인
  • 커밋 메시지 규칙 준수
  • Scene 충돌 없음

💬 기타 사항

  • 리뷰어에게 공유하고 싶은 내용

GPGS 로그인 수정
로그인 관련 버그 픽스

자음 과 모음 만으로 닉네임 설정 불가
Copy link
Collaborator

@ParkMinSung2580 ParkMinSung2580 left a comment

Choose a reason for hiding this comment

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

확인했습니다

@rnrgll rnrgll self-requested a review October 16, 2025 05:47
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

Successfully merging this pull request may close these issues.

2 participants