Initial: 회원 관련 초기 설정#12
Merged
LeeGukgeon merged 2 commits intodevelopfrom Dec 12, 2024
Hidden character warning
The head ref may contain hidden characters: "11-4-\ud68c\uc6d0-\uad00\ub9ac"
Merged
Conversation
- Spring Security 의존성 추가 - JWT 관련 의존성 추가 (jjwt-api, jjwt-impl, jjwt-jackson) - SecurityConfig 클래스 구성 - User 엔티티 정의 - Controller, Service, Repository 계층 구조 설정 Resolves: #11
kingkang85
requested review from
KIM9909,
LeeGukgeon,
MinFe4990,
uni-zun and
yunjaeeun
KIM9909
reviewed
Dec 12, 2024
MinFe4990
reviewed
Dec 12, 2024
Contributor
|
security관련 설정이 적절하게 된것같네요 |
uni-zun
reviewed
Dec 12, 2024
Contributor
uni-zun
left a comment
uni-zun
left a comment
There was a problem hiding this comment.
@EnableWebSecurity 는 어떤 역할일까요 ?
Contributor
Author
스프링 시큐리티의 웹 보안 설정을 위해 필요한 어노테이션입니다! 제가 SecurityConfig 클래스에 정의한 URL 접근 권한, 인증 방식, CSRF 설정 등이 저희 애플리케이션에 실제로 적용되기 위해 필요한 친구랍니다~! |
Contributor
|
제가 임시로 만든 UserRepository랑 출돌나서 수정해놨습니다. |
KIM9909
added a commit
that referenced
this pull request
Dec 13, 2024
코드 순서 수정, 403 forbidden 에러 해결 Resolves: #12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
📝작업 내용
의존성 설정
보안 설정
Entity 정의
🧐의문점
회원 탈퇴 시에 탈퇴 날짜가 꼭 필요한가요?? 탈퇴 시에 DB에서 유저 정보가 삭제된다면, 굳이 탈퇴 날짜가 필요 없을 것 같아서 User 엔티티 쪽에 일단 주석으로 처리해 두었는데, 의견 주시면 감사드리겠습니다..!