Feat/#8 login #10

sunwoo1256 · 2025-05-13T12:32:05Z

Spring Security 설정과 JWT Token을 발급해주는 로그인 기능을 구현했습니다.
email, password, name을 받아 database에 member를 추가하는 회원가입 기능을 구현했습니다.

이후 Redis를 이용한 refresh token 관리 기능을 추가 예정입니다.
member table의 채워지지 않은 값들과 member_preference table에서 회원의 정보들을 받아 저장해야 하므로 이 값들을 받는 회원 프로필 설정 기능을 추가할 예정입니다.

Co-authored-by: SungHo Jung <65662139+SeongHo5356@users.noreply.github.com>

…n SecurityConfig

SeongHo5356

우선 구현하느라 고생하셨습니다. 스프링 시큐리티는 다시봐도 구현할 게 많네요. 컨벤션도 잘 지키고, dto도 다 record타입으로 만들어서 사용하시고 노력한 부분이 많이 보여서 좋았습니다~

함수 네이밍에 좀 더 신경쓰고, @Valid를 활용하면 좀 더 가독성 좋고 안정적인 코드가 될 수 있을 것 같습니다.

SeongHo5356 · 2025-05-14T19:03:18Z

build.gradle

+	// JWT
+	implementation 'io.jsonwebtoken:jjwt-api:0.11.5'
+	implementation 'io.jsonwebtoken:jjwt-impl:0.11.5'
+	implementation 'io.jsonwebtoken:jjwt-jackson:0.11.5'


추후에 목적에 따라 gradle을 나눠볼 수도 있을 것 같아요!
아래 참고블로그 남겨놓을게요
gradl 분리

새로운 정보를 알게 되었습니다. 감사합니다!

SeongHo5356 · 2025-05-14T19:14:42Z

src/main/java/org/groomUniv/meet/common/security/CustomUserDetailsService.java

+    public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException {
+        return memberRepository.findByEmail(username)
+                .map(this::createUserDetails)
+                .orElseThrow(() -> new UsernameNotFoundException("User not found"));


Suggested change

.orElseThrow(() -> new UsernameNotFoundException("User not found"));

.orElseThrow(() -> new CoreException(GlobalErrorType.MEMBER_NOT_FOUND)

기존에 정의해놓은 exception 클래스를 활용할 수도 있을 것 같아요

한번 활용해 보겠습니다!!

SeongHo5356 · 2025-05-14T19:19:56Z

src/main/java/org/groomUniv/meet/common/security/JwtAuthenticationFilter.java

+import static org.aspectj.weaver.tools.cache.SimpleCacheFactory.path;
+
+@RequiredArgsConstructor
+public class JwtAuthenticationFilter extends GenericFilterBean {


Suggested change

public class JwtAuthenticationFilter extends GenericFilterBean {

public class JwtAuthenticationFilter extends OncePerRequestFilter {

GenericFilterBean: 한 번의 클라이언트 요청에 대해 여러번 요청하게 되어 자원 낭비가 발생할 수 있다.
OncePerRequestFilter: doFilterInternal 메서드를 구현하여 사용자의 요청 당 한 번만 실행되는 필터를 만들 수 있다.

감사합니다!!

SeongHo5356 · 2025-05-14T19:21:14Z

src/main/java/org/groomUniv/meet/common/security/JwtAuthenticationFilter.java

+        String token = resolveToken((HttpServletRequest) servletRequest);  // header에서 jwt 토큰을 추출
+        if (token != null && jwtTokenProvider.validateToken(token)) {   // 토큰이 존재하고 유효할 경우
+            Authentication authentication = jwtTokenProvider.getAuthentication(token);  // 토큰에서 Authentication 객체를 얻어
+            SecurityContextHolder.getContext().setAuthentication(authentication);   // Spring SecurityContext에 저장
+        }
+        filterChain.doFilter(servletRequest, servletResponse);  // 다음 필터로 요청을 전달(이걸 호출하지 않으면 요청이 중간에서 끊겨버려서 다음 단계 진행 불가)
+    }


이렇게 하면 유효하지 않아도 다음 핉터로 전달되는거 아닌가요?

그런 문제가 발생할 것 같습니다. 유효하지 않을 경우 오류를 return하는 부분을 추가해야 할 것 같습니다!

SeongHo5356 · 2025-05-14T19:22:36Z

src/main/java/org/groomUniv/meet/common/security/JwtAuthenticationFilter.java

+        System.out.println(path);
+        // 인증이 필요 없는 경로는 필터 패스
+        if (NO_AUTH_PATHS.contains(path)) {
+            System.out.println("No auth needed for path: " + path);


Suggested change

System.out.println("No auth needed for path: " + path);

log.debug("No auth needed for {}", path);

대량의 로그 발생시 서버에 악영향을 주는 Sysout을 지양해야 한다는 사실을 알게됐습니다. 감사합니다!!

SeongHo5356 · 2025-05-14T19:36:54Z

src/main/java/org/groomUniv/meet/meeting/entity/Meeting.java

+@Entity
+
+@NoArgsConstructor
+
+public class Meeting extends BaseEntity {


이런 데는 혹시 왜 줄바꿈이 있나요?

수정 해 놓겠습니다!

SeongHo5356 · 2025-05-14T19:40:24Z

src/main/java/org/groomUniv/meet/oauth/controller/MemberController.java

+    private final MemberService memberService;
+
+    @PostMapping("/login")
+    public ApiResponse<?> login(@RequestBody LoginRequest loginRequest){


Suggested change

public ApiResponse<?> login(@RequestBody LoginRequest loginRequest){

public ApiResponse<?> login(@Valid @RequestBody LoginRequest loginRequest){

요청 바디의 유효성 검사를 먼저 할 수도 있을 것 같네요
@Valid 참고자료

감사합니다! dto와 연관지어 사용해보겠습니다!

SeongHo5356 · 2025-05-14T19:40:42Z

src/main/java/org/groomUniv/meet/oauth/controller/MemberController.java

+    }
+
+    @PostMapping("/signup")
+    public ApiResponse<?> signUp(@RequestBody SignUpRequest signUpRequest){


Suggested change

public ApiResponse<?> signUp(@RequestBody SignUpRequest signUpRequest){

public ApiResponse<?> signUp(@Valid @RequestBody SignUpRequest signUpRequest){

SeongHo5356 · 2025-05-14T19:42:21Z

src/main/java/org/groomUniv/meet/oauth/dto/LoginRequest.java

+        @Schema(description = "이메일", example = "abc1234@khu.ac.kr")
+        String email,
+
+        @Schema(description = "비밀번호", example = "1234")
+        String password


Suggested change

@Schema(description = "이메일", example = "abc1234@khu.ac.kr")

String email,

@Schema(description = "비밀번호", example = "1234")

String password

@Null

@Schema(description = "이메일", example = "abc1234@khu.ac.kr")

String email,

@Null

@Schema(description = "비밀번호", example = "1234")

String password

혹시 로그인 기능이니 @notblank를 한번 사용해봐도 괜찮겠죠..?!

넵~ 공부해보시고 쓰고싶으신 거 사용하시면 될 것 같아요

SeongHo5356 · 2025-05-14T19:42:47Z

src/main/java/org/groomUniv/meet/oauth/dto/SignUpRequest.java

+        @Schema(description = "이름", example = "홍길동")
+        String name
+) {
+    public Member toEntity(String encodedPassword) {


Suggested change

public Member toEntity(String encodedPassword) {

public Member toMember(String encodedPassword) {

toMember로 하면 훨씬 명확해질 것 같습니다. 감사합니다!

lemonson03 · 2025-05-21T07:54:29Z

src/main/java/org/groomUniv/meet/common/security/JwtTokenProvider.java

+    private long refreshTokenExpirationTime;
+
+    //application.yml에 저장한 secret 값을 가져와서 key에 저장하기
+    @PostConstruct      // 스프링 컨테이너가 빈을 생성하고, 모든 의존성 주입이 끝난 뒤 자동으로 호출되는 메서드, 주입된 값을 안전하게 사용 가능


@PostConstruct로 하면 Value 값이 잘 들어온후 key값 사용하는거라 key에서 null이 안들어오겠지만
public JWTUtil(MemberRepository memberRepository,
@value("${spring.jwtsecretkey}") String secretKey) {
this.memberRepository = memberRepository;
if (secretKey == null || secretKey.isEmpty()) {
throw new IllegalArgumentException("spring.jwtsecretkey 이 설정되어야 합니다");
}
this.jwtParser = Jwts.parserBuilder()
.setSigningKey(Keys.hmacShaKeyFor(secretKey.getBytes()))
.build();
} 생성자 주입할 때 잘 적어줘도 오류가 안나긴 하더라고요 빈 객체 생성 시점에서 생기는거라 생성자 주입에 key 쓰는거 다 넣어주면 괜찮을 거 같아요

lemonson03 and others added 23 commits April 29, 2025 13:28

feat#3/basic-entity

9ba9622

feat#3/basic-repository

f141bee

feat#3/basic-repo2,report

1a4c07a

Update src/main/java/org/groomUniv/meet/blinddate/entity/BlindDate.java

894d6a8

Co-authored-by: SungHo Jung <65662139+SeongHo5356@users.noreply.github.com>

feat: (#3) Accept Comment

5734591

feat : (#3) resolve conflicts

1ddd3c9

feat: (#8) add spring security and jwt dependency

808585a

feat: (#8) add JwtToken.java DTO file

ff54eda

feat: (#8) add JwtTokenProvider

6144095

feat: (#8) add JwtAuthenticationFilter

40b9c53

feat: (#8) add SecurityConfig

a87b064

feat: (#8) register PasswordEncoder and AuthenticationManager beans i…

4ee30b3

…n SecurityConfig

feat: (#8) add role

8675b11

feat: (#8) add MemberService Interface and MemberServiceImpl

a01b363

feat: (#8) add CustomUserDetails and CustomUserDetailsService

36aef95

feat: (#8) add createUserDetails

613c9b4

feat: (#8) add login

7efa825

feat: (#8) add .csrf(AbstractHttpConfigurer::disable)

196ccfc

feat: (#8) add NO_AUTH_PATHS

3913885

feat: (#8) add signUp

aa6efbe

refactor: (#8) generateToken refactoring

c68e927

feat: (#8) add name in SignUp DTOs

ca728ac

feat: (#8) add ErrorMessage

2c80889

sunwoo1256 linked an issue May 13, 2025 that may be closed by this pull request

FEAT: 로그인 기능 구현 #8

Open

sunwoo1256 requested review from SeongHo5356 and lemonson03 May 14, 2025 10:22

SeongHo5356 requested a review from runasy-koonta May 14, 2025 18:59

SeongHo5356 reviewed May 14, 2025

View reviewed changes

sunwoo1256 added 2 commits May 18, 2025 02:27

style: (#8) remove unnecessary spaces in Meeting entity annotations

01811a9

feat: (#8) add MEMBER_NOT_FOUND exception

7a88dc8

sunwoo1256 added 4 commits May 18, 2025 02:30

feat: (#8) add handling for invalid token case

b43587f

feat: (#8) modify token expiration configuration

8b51cee

feat: (#8) add validation annotations to DTO

73f0662

refactor: (#8) rename method [toEntity] to [toMember]

88c9401

sunwoo1256 requested a review from SeongHo5356 May 17, 2025 18:41

lemonson03 reviewed May 22, 2025

View reviewed changes

	.orElseThrow(() -> new UsernameNotFoundException("User not found"));
	.orElseThrow(() -> new CoreException(GlobalErrorType.MEMBER_NOT_FOUND)

	public class JwtAuthenticationFilter extends GenericFilterBean {
	public class JwtAuthenticationFilter extends OncePerRequestFilter {

	System.out.println("No auth needed for path: " + path);
	log.debug("No auth needed for {}", path);

	public ApiResponse<?> login(@RequestBody LoginRequest loginRequest){
	public ApiResponse<?> login(@Valid @RequestBody LoginRequest loginRequest){

	public ApiResponse<?> signUp(@RequestBody SignUpRequest signUpRequest){
	public ApiResponse<?> signUp(@Valid @RequestBody SignUpRequest signUpRequest){

	public Member toEntity(String encodedPassword) {
	public Member toMember(String encodedPassword) {

Feat/#8 login #10

Are you sure you want to change the base?

Feat/#8 login #10

Uh oh!

Conversation

sunwoo1256 commented May 13, 2025

Uh oh!

SeongHo5356 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!

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants