#15,16 - Security 설정, 회원가입 개발 #34

sectionr0 · 2024-07-16T16:13:19Z

1. 🔗 관련 이슈

2. 📄 구현한 내용 또는 수정한 내용

Spring Security 설정
JWT 개발
회원가입 개발
Redis 모듈 추가

3. 🙌 추가적으로 알리고 싶은 내용

jwt yml 값 추가되었습니다
지금 수정 후 VoteServiceTest 에서 에러가 나고 있더라고요.. 아마 User 쪽이 변경되어서 그런것같기도 한데 제가 수정하기에는 양이 많아서, 재연님이 나중에 한번 봐주면 좋을 것 같아요!
(테스트 실패가 나고 있음)

4. 🙄 TODO / 고민하고 있는 것들

아직 AuthService 테스트 코드를 작성하지 않았는데, 추후 작성 후 PR 새로 올릴게요!ㅜㅜ
(재연님 테스트 코드 잘 보고 배우고 있습니다 ㅎㅎ)

5. ✅ 배포 Checklist

SecretKey 를 업데이트 해주세요
본인을 Assign 해주세요.
본인을 제외한 백엔드 개발자를 리뷰어로 지정해주세요.

- email, password, role, socialEmail 추가

- 인증과 권한 부여를 위해 사용자 정보를 캡슐화

유저와 연결 끊도록 설정

kpeel5839

양이 진짜 많네요ㅋㅋㅋㅋ 고생했어요! 아무래도 기한이 기한인 만큼 코멘트 간단하게 남겼습니다. 한번 봐주시면 감사할 것 같아요

kpeel5839 · 2024-07-17T00:28:26Z

app/src/main/kotlin/com/wespot/auth/AuthController.kt

+import org.springframework.web.bind.annotation.RestController
+
+@RestController
+@RequestMapping("/api/v1/auth")


자기야 이거 api 그냥 넣을까요?

아 ㅋㅋ 저거 안뺐네요,, 편한대로?!

일단 빼시죵!

kpeel5839 · 2024-07-17T01:10:41Z

app/src/main/kotlin/com/wespot/auth/AuthController.kt

+        val response = authService.socialAccess(request)
+
+        return if (response is SignUpResponse) {
+            ResponseEntity.status(HttpStatus.ACCEPTED).body(response)


이거는 아직 사용자가 소셜로그인만 하고 회원가입을 안한 상태인가요?

네 맞아요 이 이후에 회원가입 진행하게돼요!

kpeel5839 · 2024-07-17T01:11:05Z

app/src/main/kotlin/com/wespot/auth/AuthController.kt

+    fun signIn(
+        @RequestBody request: AuthLoginRequest
+    ): Any {
+


자기야 Function 내부에는 위, 아래 공백 안두어도 괜찮을 것 같아요!

위 아래 공백은, 지금 작업하시기에는 시간이 걸리니, 차차 제거해나가도록 하시죠!

kpeel5839 · 2024-07-17T01:11:54Z

app/src/main/kotlin/com/wespot/config/security/CustomUrlFilter.kt

+    private val validUrlPatterns = listOf(
+        "/health",
+        "/",
+        "/api/v1/auth/reissue",
+        "/api/v1/auth/login",
+        "/api/v1/auth/signup",
+        "/api/v1/auth/revoke",
+    )


앞으로 여기다가 url 추가하면 되겠네요?

네 맞아요 근데 이것도 조금 불편할 것 같기도 해서 개선해보는거 나중에 시도해볼게요!

뭔가 메타데이터 가져와서 하는 방법이 있을 것 같은데

아니면 리플렉션으로도 가능할 것 같기도 하고오옹

kpeel5839 · 2024-07-17T01:12:36Z

app/src/main/kotlin/com/wespot/config/security/CustomUrlFilter.kt

+    private fun handleInvalidUrl(
+        request: HttpServletRequest,
+        response: HttpServletResponse
+    ) {
+
+        response.contentType = MediaType.APPLICATION_JSON_VALUE
+        response.characterEncoding = UTF_8.name()
+        response.status = HttpStatus.NOT_FOUND.value()
+
+        val body = objectMapper.writeValueAsString(
+            ProblemDetail.forStatusAndDetail(
+                HttpStatus.NOT_FOUND,
+                NoSuchFieldException("잘못된 URL입니다.").message!!,
+            ).apply {
+                type = URI.create("/errors/not-found")
+                instance = URI.create(request.requestURI)
+            }
+        )
+
+        response.writer.write(body)
+
+    }
+}


키야 좋네요 👍

kpeel5839 · 2024-07-17T01:36:29Z

domain/src/main/kotlin/com/wespot/user/User.kt

+    fun withdraw() =
+        User(
+            id = id,
+            email = "",
+            password = "",
+            name = "",
+            introduction = "",
+            role = Role.GUEST,
+            schoolId = schoolId,
+            grade = grade,
+            groupNumber = groupNumber,
+            profile = profile,
+            fcm = fcm,
+            setting = setting,
+            social = Social(
+                socialEmail = "",
+                socialId = "",
+                socialType = social.socialType,
+                socialRefreshToken = ""
+            ),
+            userConsent = userConsent,
+            createdAt = createdAt,
+            updatedAt = LocalDateTime.now(),
+            withdrawAt = LocalDateTime.now(),
+        )


기가 막히는군요

kpeel5839 · 2024-07-17T01:38:29Z

infrastructure/mysql/src/main/kotlin/com/wespot/auth/RefreshTokenPersistenceAdapter.kt

+    override fun findByUserId(userId: Long): RefreshToken? {
+        return refreshTokenJpaRepository.findByUserId(userId)
+            ?.let { RefreshTokenMapper.mapToDomainEntity(it) }
+    }


조회만 있는 부분은 Transactional 야무지게 빼주셨네요 👍

kpeel5839 · 2024-07-17T01:39:01Z

infrastructure/mysql/src/main/kotlin/com/wespot/auth/RefreshTokenPersistenceAdapter.kt

+    override fun deleteByUserId(userId: Long) {
+        refreshTokenJpaRepository.deleteByUserId(userId)
+    }


요거는 Transactional 붙여줘야 하지 않을까요?

빼먹었네요..!! 추가하겠습니다!

Transactional 추가해주시면 감사할 것 같아요!

kpeel5839 · 2024-07-17T01:39:32Z

infrastructure/mysql/src/main/kotlin/com/wespot/auth/RefreshTokenPersistenceAdapter.kt

+        if (findByRefreshToken != null) {
+            return RefreshTokenMapper.mapToDomainEntity(findByRefreshToken)
+        }


얼리 리턴 좋습니다.

kpeel5839 · 2024-07-17T01:40:35Z

infrastructure/mysql/src/main/kotlin/com/wespot/user/entity/UserJpaEntity.kt

+    @OneToOne(fetch = FetchType.LAZY, cascade = [CascadeType.PERSIST])
+    @JoinColumn(name = "profile_id", foreignKey = ForeignKey(name = "fk_users_profile_id"))
+    val profile: ProfileJpaEntity,

-    @field: NotNull
-    val groupNumber: Int,
+    @OneToOne(fetch = FetchType.LAZY, cascade = [CascadeType.PERSIST])
+    @JoinColumn(name = "fcm_id", foreignKey = ForeignKey(name = "fk_users_fcm_id"))
+    val fcm: FCMJpaEntity,


이거 안지웠네요?!

rebase하다가 안지웠나봐요 ㅋㅋ.. 그래서 지웠습니다.

ㅋㅋㅋㅋ 알러뷰

아직 안지워진 것 같은데!! 내가 잘못보고 있는건가요?!

kpeel5839

기천 자기야 고생했어요. 너무 많네 우리 작업

sectionr0 added 24 commits July 16, 2024 20:34

chore: Spring Security gradle 추가

eaef785

feat: User 도메인 정보 추가

e52037e

- email, password, role, socialEmail 추가

feat: PrincipalDetail 설정

fa58236

- 인증과 권한 부여를 위해 사용자 정보를 캡슐화

feat: 시큐리티 필터 설정 및 JWT 개발

8ee5950

feat: refreshToken 개발

bf66b7c

feat: JWT Provider 개발

3bff1a2

refactor: BaseEntity 변경

da78e47

refactor: School 도메인 리펙토링

6846bd5

feat: School Repositroy, mapper 개발

92f29c8

feat: Profile Repositroy, mapper 개발

66f6489

refactor: Setting, FCM 도메인 수정

1bc7c8e

feat: UserConsent 도메인 추가

412af65

refactor: Social 도메인 수정

6efb850

refactor: User 도메인 수정 및 Role 추가

8f61085

feat: Auth Dto 설정

8cd802b

refactor: School 도메인 리팩토링

9c8c9ac

유저와 연결 끊도록 설정

refactor: RefreshToken create, update 설정 및 서비스 추가

35659a6

feat: Redis 모듈 추가, 설정 및 AuthDate 저장

e8056ed

refactor: DTO 폴더 경로 추가

e17f041

feat: SecurityUtils 추가

adab3e5

feat: AuthService 추가

bb46e8d

feat: Auth Controller 추가 및 url 설정 변경

30c01b2

refactor: rebase 후 코드 리팩토링

dbc670f

test: service Test 코드 작성

b8f61d4

sectionr0 self-assigned this Jul 16, 2024

sectionr0 added 2 commits July 17, 2024 01:21

fix: baseEntity notnull 조건 제거

d71ec8b

fix: VoteServiceTest 에러 주석 처리

1de8e82

sectionr0 requested review from kpeel5839 July 16, 2024 16:22

sectionr0 added the 🌱기능🌱 새로운 기능을 추가해요 ! label Jul 16, 2024

sectionr0 added 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🥭망고🥭 24기 조기천 labels Jul 16, 2024

kpeel5839 added 5 commits July 17, 2024 09:32

test: 주석 제거

c4b4000

chore: 서브모듈에 jwt 추가

810f5af

refactor: gitmodules 수정 이후, 이전 backend-submodule 삭제

2acf2bd

refactor: submoudle 이름 config로 변경

aa52eac

refactor: LocalDateTime.now() 모킹

6d73bee

kpeel5839 reviewed Jul 17, 2024

View reviewed changes

sectionr0 added 3 commits July 17, 2024 12:52

refactor: 401에러 예외 메시지 설정, cors 주소, delete에 트랜잭션 추가

991d491

refactor: api prefix 제거

652bf3f

refactor: refreshToken test에 있는 createdAt 수정

8f9dc12

kpeel5839 approved these changes Jul 17, 2024

View reviewed changes

github-actions bot added the 🌟머지 해주세요🌟 머지해주셔! label Jul 17, 2024

sectionr0 merged commit b0983e5 into develop Jul 17, 2024
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#15,16 - Security 설정, 회원가입 개발 #34

#15,16 - Security 설정, 회원가입 개발 #34

sectionr0 commented Jul 16, 2024 •

edited

Loading

kpeel5839 left a comment

kpeel5839 Jul 17, 2024

sectionr0 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

sectionr0 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

sectionr0 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

sectionr0 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

sectionr0 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 Jul 17, 2024

kpeel5839 left a comment

#15,16 - Security 설정, 회원가입 개발 #34

#15,16 - Security 설정, 회원가입 개발 #34

Conversation

sectionr0 commented Jul 16, 2024 • edited Loading

1. 🔗 관련 이슈

2. 📄 구현한 내용 또는 수정한 내용

3. 🙌 추가적으로 알리고 싶은 내용

4. 🙄 TODO / 고민하고 있는 것들

5. ✅ 배포 Checklist

kpeel5839 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kpeel5839 left a comment

Choose a reason for hiding this comment

sectionr0 commented Jul 16, 2024 •

edited

Loading