feat(host) : 호스트 권한 검증 aop

[ImNM]

개요

• close 🚀 [feature] 호스트 검증 aop #249

작업사항

• 호스트의 권한을 검증할 수 있는 aop 를 개발 했습니다.
• 중복된소스를 줄이고, 명시적으로 선언하는게 눈에도 더 잘보일것 같아서 만들었습니다.

제일 중요한 부분은 아래 부분입니다,

// HostQualification
    MASTER((userId, host) -> host.validateMasterHostUser(userId)),
    MANAGER((userId, host) -> host.validateSuperHostUser(userId)),
    GUEST((userId, host) -> host.validateActiveHostUser(userId));

// biConsumer로 실행시킬 함수 선언
private final BiConsumer<Long, Host> consumer;

사용방법

• before

    @Transactional
    public EventResponse execute(Long eventId, UpdateEventDetailRequest updateEventDetailRequest) {
        final User user = userUtils.getCurrentUser();
        final Long userId = user.getId();
        final Event event = eventAdaptor.findById(eventId);
        hostService.validateSuperHostUser(event.getHostId(), userId);

• after

@HostRolesAllowed("MANAGER")
public EventResponse execute(Long eventId, UpdateEventDetailRequest updateEventDetailRequest) {

위 세가지 값을 명시적으로 적어주시면 됩니다.

• 트랜잭션 내부에서 캐시하는 효과를 볼수 있도록
• HostRoleTransaction 통해서 aop 안에 한 뎁스 더 들어가서 repository에서 값을 불러왔습니다.

변경로직

• 내용을 적어주세요.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
263 Code Smells

16.3% Coverage
0.0% Duplication

[gengminy]

AOP 구현 잘봤습니다

[gengminy]

메소드 파라미터에 eventId 가 없는 경우에도 검증이 되나요?

[ImNM]

네넹 IllegalArgumentException 입니당

[gengminy]

요기서 되는건가

[kim-wonjin]

잘써먹겠습니다 굿굿😸

[sanbonai06]

좋네요 적용해서 리팩 하겠습니다