toss / nestjs-aop

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

[BUG]: Decorating order of AopDecorator is not guaranteed #7

Closed WhiteKiwi closed 1 year ago

WhiteKiwi commented 1 year ago

Describe the bug

It is not known which will be decorated first, Log or Cache, in the following situations:

class SomeService {
  @Log()
  @Cache()
  doSomethingA() { ... }

  @Log()
  @Cache()
  doSomethingB() { ... }
}

Expected behavior

To Reproduce

Possible Solution

Additional context

WhiteKiwi commented 1 year ago

It will be resolved in v2.