Skip to content

fix: [auth] passwordEncoder 중복 선언 삭제 - #44

Merged
d6nggyun merged 1 commit into
mainfrom
dev
Apr 9, 2026
Merged

d6nggyun merged 1 commit into
mainfrom
dev

Conversation

@d6nggyun

@d6nggyun d6nggyun commented Apr 9, 2026 •

Copy link
Copy Markdown
Collaborator

#️⃣ 연관된 이슈

📝 작업 내용

[auth] passwordEncoder 중복 선언 삭제

스크린샷 (선택)

💬 리뷰 요구사항(선택)

Summary by CodeRabbit

릴리스 노트

  • 리팩토링
    • 인증 설정의 코드 구조를 최적화했습니다.
    • 불필요한 설정 요소를 제거하여 코드베이스를 정리했습니다.

@d6nggyun d6nggyun self-assigned this Apr 9, 2026
@d6nggyun d6nggyun added the 🐞 BugFix 버그 발생 또는 해결 label Apr 9, 2026
@coderabbitai

coderabbitai Bot commented Apr 9, 2026 •

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8462a526-fd73-4f73-9ff9-7f0a78cd9028

📥 Commits

Reviewing files that changed from the base of the PR and between 3df4b5c and 9b3a92e.

📒 Files selected for processing (1)
  • codin-auth/src/main/java/inu/codin/auth/config/SecurityConfig.java

📝 Walkthrough

개요

보안 설정 파일에서 OAuth2 로그인 핸들러 임포트를 재정렬하고, 중복 임포트 라인을 제거했습니다. PasswordEncoder 빈 정의 및 관련 임포트를 제거하여 설정을 정리했습니다.

변경 사항

Cohort / File(s) Summary
SecurityConfig 정리
codin-auth/src/main/java/inu/codin/auth/config/SecurityConfig.java
OAuth2 로그인 핸들러 임포트 재정렬, 중복 임포트 제거, passwordEncoder() 공개 빈 메서드 제거, 불필요한 공백 라인 삭제.

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~5 minutes

시

🐰 빈의 정원에서 잡초를 뽑아내니,
암호화기는 다른 곳으로 옮겨가고,
임포트는 깔끔하게 줄을 맞춰,
설정파일이 더 가벼워졌네! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@d6nggyun
d6nggyun merged commit d4ab720 into main Apr 9, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix 버그 발생 또는 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant