toss / nestjs-aop

A way to gracefully apply AOP to nestjs
MIT License
213 stars 24 forks source link

[BUG]: 특정 class 를 dynamic module 로 재사용하는 경우 의도하지 않은 동작을 합니다. #18

Closed kys0213 closed 1 year ago

kys0213 commented 1 year ago

Describe the bug

아래와 같이 재사용되는 service 가 있다고 가정할 때, 의존성 주입은 A service 지만 aop 데코레이터를 거치면서 b service 의 함수를 참조합니다.

class SomeService {
  constructor(private requester: Requester) {}

  @Cacheable()
  executr() {
    // SomeProvider 를 재사용하며, 서로다른 모듈에서 requester 를 별도로 주입해줌.
    this.requester.request()
  }
}

Expected behavior

기대하는 동작은 aop 데코레이터를 거치더라도 주입된 service 를 사용해야합니다.

To Reproduce

Possible Solution

Additional context