Skip to content

Conversation

vectorch9
Copy link
Member

@vectorch9 vectorch9 commented Jun 23, 2023

📌Linked Issues

✏Change Details

의존성 추가 및 변경

  • Swagger, Security 관련 설정을 추가하였습니다.
    • Swagger의 경우 이전 프로젝트에서 사용하던 버전에 취약점 관련 알림이 계속 뜨는게 신경쓰여서 3.0.0버전으로 올렸습니다.
    • Security의 경우도 이전 프로젝트의 @Deprecated관련 사항을 이번 기회에 확인 하고 싶어 버전을 올렸습니다. -> 이 과정에서 전체 버전을 맞추는게 좋을 것 같아 스프링 버전을 2.5.4에서 2.7.5로 올렸습니다.
      • 사실 큰 의미없는 작업이긴 하지만, 프로젝트를 새로 시작하는 김에 다른 버전도 한번 사용해보고 싶었습니다.

유효성 검증 추가

  • 이전 작업에서 누락된 RegisterRequest에 대한 유효성 검증을 추가하였습니다.
@ApiModelProperty(value = "유저네임", example = "dlwlrma", required = true)
@NotBlank(message = "username을 입력해주세요")
@Size(min = 4, max = 12, message = "사용자 이름은 4문자 이상 12문자 이하여야 합니다")
@Pattern(regexp = "^[0-9a-zA-Z]+$", message = "username엔 대소문자, 숫자만 사용할 수 있습니다.")
private String username;
  • 현재 Request클래스는 위와 같은 형태를 가집니다.
  • 실패 message또한 자바 코드상에 포함되어있어 가독성, 유지보수 측면에선 별로 안좋다고 생각하는데 어떻게 생각하시나요?
    • 개선 방법으론 스프링 메시지(ex. messages.properties) 기능을 이용하여 실패 메시지를 외부로 분리하는게 어떨지 생각하고 있습니다. 이 경우 GlobalExceptionHandler에서 메시지 처리를 위해 약간의 로직 수정이 필요합니다!
    • 실무에선 주로 어떻게 처리하나요? 🤔

<!-
✅ 추가로 전달할 내용이 있다면 적어주세요.
-->

💬Comment

📑References

✅Check List

  • 추가한 기능에 대한 테스트는 모두 완료하셨나요?
  • 코드 정렬(Ctrl + Alt + L), 불필요한 코드나 오타는 없는지 확인하셨나요?

@vectorch9 vectorch9 added the enhancement New feature or request label Jun 23, 2023
@vectorch9 vectorch9 requested a review from seonpilKim June 23, 2023 12:00
@vectorch9 vectorch9 self-assigned this Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant