[Suggestion] - add simplified class for interceptor

Hi, I think it would be great to have a simplified version of the interceptor mechanism, which will use the current implementation.

I suggest to create a base interceptor class, which will look like so:

export abstract class BaseInterceptor implements NestInterceptor {
  protected onBefore(request?: any) {
  }

  protected onAfter(data?: any) {
  }

  intercept(dataOrRequest: any, context: ExecutionContext, stream$: Observable<any>): Observable<any> {
    this.onBefore(dataOrRequest);

    return stream$.map((data) => {
      return this.onAfter(data);
    });
  }
}

And here is how the LoggingInterceptor will look:

@Interceptor()
export class LoggingInterceptor extends BaseInterceptor {
  onBefore(request: any) {
    console.log('Before...');
  }

  onAfter(data) {
    console.log('After...');

    return data;
  }
}

The benefit of this implementation is the simplified (in my opinion) API that it brings, WDYT?

Note: it would be even better if there we different mechanism for interception when it will require an interceptor to implement the onBefore and onAfter interfaces. This way inheritance won’t be needed.

[vlio20]

@wbhob, @kamilmysliwiec can I PR?

[lock[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.