Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature]: 탈퇴한 유저, 이용제한, 영구정지 당한 유저의 접근을 막는다. #74

Closed
2 tasks done
kpeel5839 opened this issue Aug 5, 2024 · 1 comment
Assignees
Labels
🌱기능🌱 새로운 기능을 추가해요 ! 🏋️매튜🏋️ 24기 김재연

Comments

@kpeel5839
Copy link
Contributor

kpeel5839 commented Aug 5, 2024

Epic/Story Ticket Number

None

작업 사항

기천짱 제가 생각한 방법은 현재 Security Utils로 유저를 다 찾아오고 있잖아요!
그러면 여기서 탈퇴한 유저, 이용제한, 영구정지 당한 유저인 경우 예외를 던지기만 하면 좋을 것 같아요!
기천짱 생각은 어떠신가요?

image

Todo

  • 방어 로직 작성
  • 테스트 추가

기타사항

No response

@kpeel5839 kpeel5839 added 🌱기능🌱 새로운 기능을 추가해요 ! 🏋️매튜🏋️ 24기 김재연 labels Aug 5, 2024
@kpeel5839 kpeel5839 self-assigned this Aug 5, 2024
@sectionr0
Copy link
Contributor

조회 API도 다 사용가능하고, 탈퇴의 경우는 오프라인에서 얘기했듯이 필터에서 접근을 막으면 안될 것 같아요!

아니면 권한에 따라 API 사용할 수 있는걸 나누면 좋을 것 같네용

kpeel5839 added a commit that referenced this issue Aug 11, 2024
* refactor: Message, Vote Restriction 구분

* refactor: 바꾼 스펙에 맞춰 Test 수정

* refactor: 탈퇴 로직 수정

* test: User Test 추가

* test: RestrictionService Test 추가

* refactor: 이름 변경

* refactor: 이름 변경

* refactor: 스펙 변경

* refactor: RankServiceTest 수정

* refactor: Users Test 수정

* test: RestrictionTest 추가

* test: Vote,Message Restriction Test 추가

* refactor: 스펙 postman에 맞춰 변경

* refactor: Restriction Table 분리

* test: Restriction 분리 후 테스트 보강
kpeel5839 added a commit that referenced this issue Sep 3, 2024
* refactor: 탈퇴 및 이용제한 걸린 친구는 투표를 받을 수 없도록 수정

* refactor: 쿼리 일부 최적화

* refactor: 규제를 당한 유저는 해당 서비스 사용할 수 없도록 수정

* refactor: 제재를 당한 유저는 검색을 할 수 없도록 수정

* feat: Message 관련하여 탈퇴 혹은 규제를 당한 유저가 서비스를 사용할 수 없도록 수정

* refactor: loginUserId 추가

* style: 사용하지 않는 import 제거

* refactor: EventUtils 선언

* refactor: ApplicationEventPublisher Utils로 변경

* test: 쪽지 테스트 추가

* style: 개행 제거

* test: VoteTest 추가

* test: UserSearchTest 추가

* test: classmate 찾는 jpa method 테스트 추가

* test: Test 전부 통과하도록 수정

* test: Search User Service Test 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱기능🌱 새로운 기능을 추가해요 ! 🏋️매튜🏋️ 24기 김재연
Projects
None yet
Development

No branches or pull requests

2 participants