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

fix, feat: 회원가입 및 REST API 이슈 및 오류사항 수정 #600

Merged
merged 10 commits into from
Jun 17, 2024

Conversation

KimTom89
Copy link
Collaborator

@KimTom89 KimTom89 commented Jun 14, 2024

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요. PR을 보내기 전에 모든 항목을 확인해야 합니다.

  • 동일한 업데이트/변경에 대한 다른 Pull Requests가 열려있는지 확인했습니다.
  • 테스트가 성공적으로 수행되었는지 확인했습니다.

PR 유형

어떤 유형의 PR인가요? (해당 항목에 모두 체크해주세요)

  • 버그 수정
  • 새로운 기능
  • UI/UX 개선
  • 문서내용 수정
  • 코드 의미에 영향을 주지 않는 변경사항 (오타, 서식 지정, 변수명 변경 등)
  • 코드 리팩토링 (버그 수정이나 기능 변경 없는 코드 변경)
  • 빌드 관련 변경
  • 테스트 코드 추가
  • 기타 (이유를 설명해주세요.)

변경 사항

REST API

  1. Captcha 검증기능 추가
  2. JWT
    • Refresh Token의 sub claim에 사용자 ID를 추가
    • JWT 생성 시, nbf(활성화 시간), aud(대상자) 속성 추가
    • timestamp > float -> int형 변경

회원가입/정보수정

  1. 금지단어, 욕설 필터링 추가
  2. 비밀번호 자릿수 제한 오류 수정
  3. 가입정보 유효성 검사 추가 (추천인 포함)
  4. 여분필드 조회/수정 추가
  5. 인증메일 변경 기능 추가 (템플릿, API)

기타

  1. 로그인 시, '올바르지 않은 URL입니다.' 오류 재수정
    • 잘못된 URL이더라도 로그인 될 수 있도록 수정
    • 경고창 출력(예외처리) -> 메인 페이지로 리다이렉트
  2. 자기소개 > 삭제된 css를 불러오는 코드 삭제

관련 이슈

기타 정보

KimTom89 added 10 commits June 13, 2024 15:49
- Refresh Token의 sub claim에 사용자 ID를 추가
- JWT 생성 시, nbf, aud 속성 추가
- 'validate_login_url()' 함수 처리방식 변경
    - 예외 발생 -> 메인 페이지로 리다이렉트
- 이름 > 단어 필터링 추가 (욕설)
- 아이디/닉네임 > 금지단어 필터링 개선 (대소문자 구분 X)
    - 금지단어가 포함된 경우는 처리하지 않음. (ex: 'su' 단어의 경우 'sugar'도 필터링)
- aud(토큰 대상자) 인증 오류 수정
- timestamp > float -> int로 변경
- 회원가입/수정 시 비밀번호 자릿수를 체크하지 않았던 오류 수정
- 4~20자 이내의 비밀번호만 입력 가능하도록 수정
- 프론트 단에서 주석처리되어있던 스크립트 적용
- 추천인 유효성 검사 추가
- core/exception.py > JSONException이 추가적인 매개변수를 받도록 수정
- 기존 회원가입 > 인증메일 변경 기능 추가(템플릿, API)
- 회원 가입 메일 발송 > 회원과 관리자에게 발송되는 메일을 분리
- 적용 중인 구글 Recaptcha v2, invisible에 대한 검증 기능 추가
@KimTom89 KimTom89 requested review from kagla and removed request for kagla June 17, 2024 01:56
@KimTom89 KimTom89 added bug Something isn't working feature New feature or request labels Jun 17, 2024
@KimTom89 KimTom89 merged commit 4b8d139 into master Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant