Skip to content

[Feature] API 공통 응답 구현#39

Merged
Etwashoeren merged 34 commits intodevelopfrom
feature/#35-API-Response
Mar 20, 2025
Merged

[Feature] API 공통 응답 구현#39
Etwashoeren merged 34 commits intodevelopfrom
feature/#35-API-Response

Conversation

@Etwashoeren
Copy link
Contributor

📌 개요

  • 전역 예외처리를 통한 API 공통 응답 구현 및 로그인, 회원가입, 아이디 찾기 및 비밀번호 재설정, 견적서 관련 기능 로직을 수정하였습니다.

🚀 관련 이슈

✅ 변경 사항

  • RestAPI 통신 시, 에러 사항에 관한 status 코드와 메시지 반환.
  • 로그인, 회원가입, 아이디 찾기 및 비밀번호 재설정, 견적서 관련 기능 로직 수정
  • 로그인, 회원가입, 아이디 찾기 및 비밀번호 재설정 관련 Swagger 설명 추가

📝 상세 내용

RestAPI 응답 관련

  • RestAPI 예외처리 응답 부분
    • 각 기능별로 예외처리를 진행하여 각 에러에 대한 코드와 메시지를 반환 합니다.
      ex) MEMBER_NOT_FOUND(code), 404(status), 사용자를 찾을 수 없습니다.(message)

로그인 관련

  • 로그인, 회원 가입, 이메일 인증에 대한 반환 값 수정
    • 해당 기능들에 대한 반환 값들은 보안상 응답 body에 넣는 것이 절절치 않다고 판단되어 성공하면 성공 메시지를 반환하고, 응답 body에는 data를 넣지 않았습니다.
      예시)
      "status": 200,
      "message": "회원가입 성공",
      "data": null
  • JWT 발급 로직 수정
    • 기존 발급 시에는 사용자 role에 대한 정보가 담겨 있지 않은 것을 확인하여 jwt 발급 시, 사용자 role를 파라미터 값으로 넣도록 수정하였습니다.
  • 로그인 반환 값 수정
    • 기존에도 로그인 시 accessToken만 반환하도록 하였으나 기존에는 accessToken 값만 반환했다면 이번 수정으로인해 grantType과
      accessToken값 그리고 role을 반환하도록 하였습니다.
  • 로그인 DTO 유효성 추가
    • 로그인 시에 입력 값들에 대한 유효성 검사(공백 x, 길이 및 패턴)가 진행될 수 있도록 수정하였습니다.

회원 가입 관련

  • 사용자 아이디 중복 체크 유효성 검사 추가
    • 사용자 아이디 중복 체크 시에도 영문자 및 숫자 포함 7자리 이상 15자리 이하에 대한 유효성 검사가 진행될 수 있도록 수정하였습니다.

이메일 관련

  • 이메일 인증 확인 추가
    • 회원가입 단계에서 이메일 인증 번호 일치 여부 확인 기능을 추가하였습니다.

@rimeir
Copy link
Member

rimeir commented Mar 18, 2025

p1: application.yml에서 충돌이 발생한 것 같습니다!
내일까지 변경사항 확인 및 충돌 부분 해결하고 리뷰 남기겠습니다~

Copy link
Member

@rimeir rimeir left a comment

Choose a reason for hiding this comment

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

변경사항 확인했습니다! 각 기능별 응답을 통합하고 일관된 형식으로 만들어 더욱 편리하게 작업할 수 있을 것 같아요! 수고하셨습니다~!
p3: application.yml 파일 충돌 해결했습니다! PR 전에 develop 브랜치를 최신 상태로 만들고 작업하시는 브랜치에서 리베이스를 진행하시면 충돌을 예방할 수 있습니다!

@Etwashoeren Etwashoeren self-assigned this Mar 19, 2025
@Etwashoeren Etwashoeren added enhancement New feature or request backend labels Mar 19, 2025
@Etwashoeren Etwashoeren linked an issue Mar 19, 2025 that may be closed by this pull request
@Etwashoeren
Copy link
Contributor Author

엇,, 기능에만 충실하느라 그런 부분을 너무 신경 못 썼네요,, 번거롭게 해드려 죄송합니다ㅠ PR 전 develop 브랜치 최신화 및 리베이스 진행 숙지하도록 하겠습니다!

@github-project-automation github-project-automation bot moved this to Done in PostDM Mar 20, 2025
@Etwashoeren Etwashoeren reopened this Mar 20, 2025
@Etwashoeren Etwashoeren merged commit a757567 into develop Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Feature] API 공통 응답 구현

2 participants