[#2] JWT 로그인 #26

hado0123 · 2025-11-25T05:01:51Z

작업 내용

JWT 기반 로그인 인증 기능 구현
Access Token 발급 기능 추가
비밀번호 검증 로직 구현
관련 설정(Security, Filter) 적용

테스트 내용

Postman 로그인 요청 성공 확인
유효하지 않은 비밀번호 시 예외 반환 확인
토큰 포함 요청 시 정상 인증 여부 확인
유효하지 않은 토큰, 유효기간이 지난 토큰 예외 반환 확인

추가 필요

Refresh Token 도입 및 재발급 로직 구현 예정
로그인 성공 응답 DTO 정리 필요

f-lab-saponin

고생하셨습니다👍 리뷰한번 확인부탁드려요!

f-lab-saponin · 2025-11-28T07:51:35Z

README.md

+### 🔹 방법 1) Docker MySQL + IDE(인텔리제이) 앱 실행
+
+MySQL만 Docker로 실행하고, Spring Boot 앱은 IDE에서 실행하는 방식입니다.
+
+```bash
+docker compose up -d mysql
+```
+
+### 🔹 방법 2) Docker MySQL + Docker Spring Boot 앱 실행 (전체 Docker 실행)
+
+MySQL과 Spring Boot 앱을 모두 Docker로 실행하는 방식입니다.
+```bash
+docker compose up -d mysql
+docker compose up -d --build


하나로 정해서 정리하면 좋아보여요🙂

하나로 정리 완료했습니다!

f-lab-saponin · 2025-11-28T08:27:52Z

src/main/java/com/creatorhub/constant/Role.java

+    // CREATOR는 MEMBER 권한 + CREATOR 권한을 모두 가진다
+    CREATOR(Set.of("ROLE_MEMBER", "ROLE_CREATOR"));


f-lab-saponin · 2025-11-28T08:28:27Z

src/main/java/com/creatorhub/controller/AuthController.java

+    @PostMapping("/make")
+    public ResponseEntity<Map<String, String>> makeToken(@RequestBody LoginRequest loginRequest) {


하는 일이 로그인 같은데 "/login" 해도 될것같아요 🙂

변경 완료했습니다!

f-lab-saponin · 2025-11-28T08:34:34Z

src/main/java/com/creatorhub/entity/Member.java

+    @Column(nullable = false, unique = true)
+    private String email;
+
+    @Column(nullable = false)
+    private String password;
+
+    @Column(nullable = false, length = 150)
+    private String name;


unique 는 고민해보시고
LoginRequest 에 있는 검증처럼 length 도 DB 타입에 넣어놓으면 좋을것같아요

email, password 각각 DTO에 맞춰 length추가 완료했습니다!

f-lab-saponin · 2025-11-28T08:37:11Z

src/main/java/com/creatorhub/repository/MemberRepository.java

+import java.util.Optional;
+
+public interface MemberRepository extends JpaRepository<Member, Long> {
+    Optional<Member> findByEmail(String email);


INDEX 생성하셨다면 DDL 관리를위해 이것도 ddl 파일에 기록해주세요!

f-lab-saponin · 2025-11-28T08:42:45Z

src/main/java/com/creatorhub/security/util/JWTUtil.java

+@Component
+@Slf4j
+public class JWTUtil {
+    private final SecretKey secretKey;


accessToken 과 refreshToken 의 secretKey 는 다르게 하는게 보안을 더 향상할 수 있는 방법이예요.
어떻게 만들면 좋을지 고민해보면 좋겠습니다. 🙂

secretKey 분리 완료했습니다

088c826

f-lab-saponin · 2025-11-28T08:48:01Z

src/main/java/com/creatorhub/service/MemberService.java

+    @Transactional
+    public MemberResponse signup(MemberRequest memberRequest) {


@Transactional 이 꼭 필요하면 넣으면 좋아보여요. 필요할까요?

회원가입 쪽은 나중에 초기 캐시 적립이나 다른 작업들이 묶일 수도 있을 것 같아서 일단은 @Transactional 유지해뒀어요.
이렇게 해도 괜찮을까요?

f-lab-saponin · 2025-11-28T08:48:14Z

src/main/java/com/creatorhub/service/MemberService.java

+    @Transactional
+    public void deleteMember(Long memberId) {


여기두요 🙂

회원가입과 마찬가지로 삭제시 같이 지워야 할 연관 데이터와 같은 다른 작업이 묶일수도 있다고 생각해서 일단 @Transactional 유지해 두었습니다.

f-lab-saponin · 2025-11-28T08:48:59Z

src/main/resources/application.yml

+jwt:
+  secret: ${JWT_SECRET:1234567890123456789012345678901234567890}


secretKey 노출이 안되도록 해볼까요?

.env 파일로 secretKey 및 DB 접속정보 처리했습니다

325ae16

application.yaml + gitignore 조합도 추천합니다 🙂

f-lab-saponin · 2025-11-28T08:49:49Z

src/test/java/com/creatorhub/service/MemberServiceTest.java

+import static org.assertj.core.api.Assertions.*;
+
+@SpringBootTest
+//@Transactional


이건 의도하신걸까요? 데이터를 생성하고 다시 삭제하는걸 의도하셨다면 유지해야할 것 같아요.

잠깐 DB 데이터 확인 후 주석을 푼다는 걸 깜빡했네요! 원래대로 돌려놨습니다

sonarqubecloud · 2025-12-01T10:49:05Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

f-lab-saponin

리뷰 반영해주시느라 고생하셨습니다 승인드립니다. 🙂
추가 코멘트 드린 부분 + Transactional 부분은 선택이니 고민해보시고 반영부탁드려요

hado0123 added 21 commits November 18, 2025 11:17

[#1] 회원가입

72b52b4

[#1]refactor: DTO record로 변경

b1a5c24

[#1] refactor: Member 정적 팩토리 메소드 createMember 추가

6f5d7ca

Merge remote-tracking branch 'origin/feature/1' into feature/1

a5e8238

[#1] style: 로그 구현체 Slf4j로 변경

6c1d58f

[#1] fix: 트랜잭션 범위 최소화 적용

9feb133

[#1] chore: application.properties → application.yml 변환

b049217

[#1] refactor: ARTIST → CREATOR enum 수정

b9b2f3c

[#1] feat: Creator 엔티티 및 1:1 매핑 추가

c9a77d0

[#1] feat: soft delete(deletedAt) 추가

2c64617

[#1] feat: SecurityConfig 필터 체인 설정 추가

ac0d0bd

[#1] feat: 테스트 코드 회원삭제 추가

6086813

[#1] refactor: 삭제 처리

bbe326d

[#1] chore: Dockerfile 및 docker-compose 설정 적용

07bc194

[#1] chore: Docker Compose 실행 시 테스트용 DB 생성 스크립트 추가

dc10fd1

[#1] chore: Docker MySQL 데이터 폴더(mysql-data) Git 제외

5bdfb40

[#1] chore: 디폴트 DB 연결 수정

146f4e9

[#1] feat: Member 관련 예외 클래스 정의 및 글로벌 예외 처리

130b453

[#1] chore: gradlew에 실행권한 부여

a3e6d35

[#1] docs: README에 Docker 기반 실행 방법 추가

b02c481

[#2] feat: JWT Access 토큰 발급/검증 기능 추가

daa033c

hado0123 requested a review from f-lab-saponin November 25, 2025 05:01

hado0123 self-assigned this Nov 25, 2025

[#2] refactor: Role enum 계층 구조 정의 및 JWT claims 객체화

6ebcc33

f-lab-saponin suggested changes Nov 28, 2025

View reviewed changes

hado0123 added 5 commits November 29, 2025 16:03

[#2] feat: Refresh 토큰(Redis 저장) 적용

1e376d9

[#2] docs: Readme Docker 실행 방법 수정, Redis 추가

4b13710

[#2] docs: Readme Docker 실행 방법 오타 수정

d46e911

[#2] docs: Readme Docker 실행 방법 정리 및 .env 파일 설명 추가

c545e22

[#2] refactor: email, password length 추가

bf00a94

hado0123 added 3 commits December 1, 2025 19:46

[#2] refactor: refreshToken, AccessToken key 분리

088c826

[#2] refactor: refreshToken, AccessToken key .env 파일 처리

325ae16

[#2] refactor: 트랜잭션 처리 변경

f77b304

f-lab-saponin approved these changes Dec 2, 2025

View reviewed changes

		// CREATOR는 MEMBER 권한 + CREATOR 권한을 모두 가진다
		CREATOR(Set.of("ROLE_MEMBER", "ROLE_CREATOR"));

		@PostMapping("/make")
		public ResponseEntity<Map<String, String>> makeToken(@RequestBody LoginRequest loginRequest) {

		@Transactional
		public MemberResponse signup(MemberRequest memberRequest) {

		jwt:
		secret: ${JWT_SECRET:1234567890123456789012345678901234567890}

[#2] JWT 로그인 #26

Are you sure you want to change the base?

[#2] JWT 로그인 #26

Uh oh!

Conversation

hado0123 commented Nov 25, 2025

작업 내용

테스트 내용

추가 필요

Uh oh!

f-lab-saponin left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hado0123 Dec 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hado0123 Dec 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Dec 1, 2025

Quality Gate passed

Uh oh!

f-lab-saponin left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

hado0123 Dec 1, 2025 •

edited

Loading

hado0123 Dec 1, 2025 •

edited

Loading