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] 멤버 및 시큐리티 리팩토링 #92

Merged
merged 4 commits into from
Apr 1, 2024

Conversation

ohksj77
Copy link
Member

@ohksj77 ohksj77 commented Mar 29, 2024

추가/수정한 기능 설명

  • 리팩토링 수행, 기능의 추가/수정은 없음
    • null 사용하던 코드 Optional 사용
    • Key 주입 방식 변경
    • 하드 코딩 일부 제거

특이사항

  • initializingBean 제거 관련 참고할 만한 레퍼런스

  • 개인적인 견해

    • 생성자로 주입할 수 있는 객체지향적으로 제공되는 기능을 어길 만큼 특별한 객체 초기화라고 생각되지 않음
    • 기존 방식 대로면 key가 final 도 아니며 환경변수를 멤버변수로 갖고 있을 이유가 없음에도 TokenProvider의 멤버 변수임

check list

  • 모든 단위 테스트를 돌려보고 기존에 작동하던 테스트에 영향이 없는 것을 확인했나요?
  • 추가/수정사항을 설명했나요?

@ohksj77 ohksj77 added the refactor 리팩토링 label Mar 29, 2024
@ohksj77 ohksj77 requested a review from jinjoo-lab March 29, 2024 03:58
@ohksj77 ohksj77 self-assigned this Mar 29, 2024
@ohksj77
Copy link
Member Author

ohksj77 commented Mar 29, 2024

Test Coverage

File Coverage [52.74%] 🍏
AuthService.java 52.74% 🍏
Total Project Coverage 55.51% 🍏

@ohksj77 ohksj77 merged commit 600acd3 into master Apr 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant