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

[refactor] custom acl 리팩토링 #115

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

[refactor] custom acl 리팩토링 #115

wants to merge 4 commits into from

Conversation

qogustj
Copy link
Contributor

@qogustj qogustj commented Sep 20, 2024

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

ACL 체크가 필요한 메서드에 적용할 커스텀 어노테이션 구현
AROUND 어드바이스를 사용하여 RESPONSE를 가로채고 수정하는 Aspect 정규화?
ACL 체크가 필요한 메서드에 @CustomACL 어노테이션을 적용
service에 action과 boarcode를 설정해줌
ㄴaction으로 api 맘대로 호출하는거 방지


✏️ 관련 이슈(선택 사항)

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요

ex)


코드 리뷰 받고 싶은 부분


@qogustj qogustj added the refactor Refactoring the code label Sep 20, 2024
@qogustj qogustj self-assigned this Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant