bugfix(@nestjs/common) populate multer exceptions

Author: kamilmysliwiec

CHANGELOG.md

@@ -1,5 +1,10 @@
 ## 4.6.4
 ### Bug Fixes
+- **common**: logger overrides custom logger scope [#435](https://github.com/nestjs/nest/issues/435)
+- **common**: `FileInterceptor` supports only one options at once [#429](https://github.com/nestjs/nest/issues/429)
+- **common**: support `symbol` as a token (`NestApplicationContext`)
+- **core**: fix exception handler (exceception thrown in the `done()` callback) [#431](https://github.com/nestjs/nest/issues/431)
+- **core**: incorrect HTTP response on `SyntaxError` [#430](https://github.com/nestjs/nest/issues/430)
 - **microservices**: can't select/get from context when using `NestFactory.createMicroservice` [#398](https://github.com/nestjs/nest/issues/398)
 
 ## 4.6.3

lib/common/interceptors/file.interceptor.d.ts

@@ -1,8 +1,2 @@
-import { Observable } from 'rxjs/Observable';
 import { MulterOptions } from '../interfaces/external/multer-options.interface';
-export declare function FileInterceptor(fieldName: string, options?: MulterOptions): {
-    new (): {
-        readonly upload: any;
-        intercept(request: any, context: any, stream$: Observable<any>): Promise<Observable<any>>;
-    };
-};
+export declare function FileInterceptor(fieldName: string, options?: MulterOptions): any;

lib/common/interceptors/file.interceptor.js

@@ -9,8 +9,9 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const multer = require("multer");
+const component_decorator_1 = require("../decorators/core/component.decorator");
 function FileInterceptor(fieldName, options) {
-    const Interceptor = class {
+    return component_decorator_1.mixin(class {
         constructor() {
             this.upload = multer(options);
         }
@@ -20,7 +21,6 @@ function FileInterceptor(fieldName, options) {
                 return stream$;
             });
         }
-    };
-    return Interceptor;
+    });
 }
 exports.FileInterceptor = FileInterceptor;

lib/common/interfaces/nest-application-context.interface.d.ts

@@ -9,5 +9,5 @@ export interface INestApplicationContext {
      * Makes possible to retrieve the instance of the component or controller available inside the processed module.
      * @returns T
      */
-    get<T>(metatypeOrToken: Metatype<T> | string): T;
+    get<T>(metatypeOrToken: Metatype<T> | string | symbol): T;
 }

lib/common/services/logger.service.js

@@ -9,15 +9,15 @@ class Logger {
     }
     log(message) {
         const { logger } = Logger;
-        logger.log(message, this.context, this.isTimeDiffEnabled);
+        logger.log.call(logger, message, this.context, this.isTimeDiffEnabled);
     }
     error(message, trace = '') {
         const { logger } = Logger;
-        logger.error(message, trace, this.context, this.isTimeDiffEnabled);
+        logger.error.call(logger, message, trace, this.context, this.isTimeDiffEnabled);
     }
     warn(message) {
         const { logger } = Logger;
-        logger.warn(message, this.context, this.isTimeDiffEnabled);
+        logger.warn.call(logger, message, this.context, this.isTimeDiffEnabled);
     }
     static overrideLogger(logger) {
         this.logger = logger;

lib/core/nest-application-context.d.ts

@@ -8,6 +8,7 @@ export declare class NestApplicationContext implements INestApplicationContext {
     protected contextModule: any;
     private readonly moduleTokenFactory;
     constructor(container: NestContainer, scope: NestModuleMetatype[], contextModule: any);
+    selectContextModule(): void;
     select<T>(module: Metatype<T>): INestApplicationContext;
     get<T>(metatypeOrToken: Metatype<T> | string | symbol): T;
     private findInstanceByPrototypeOrToken<T>(metatypeOrToken);

lib/core/nest-application-context.js

@@ -9,6 +9,10 @@ class NestApplicationContext {
         this.contextModule = contextModule;
         this.moduleTokenFactory = new module_token_factory_1.ModuleTokenFactory();
     }
+    selectContextModule() {
+        const modules = this.container.getModules().values();
+        this.contextModule = modules.next().value;
+    }
     select(module) {
         const modules = this.container.getModules();
         const moduleMetatype = this.contextModule.metatype;

lib/core/nest-application.d.ts

@@ -21,7 +21,6 @@ export declare class NestApplication extends NestApplicationContext implements I
     private readonly microservices;
     private isInitialized;
     constructor(container: NestContainer, express: any, config: ApplicationConfig, appOptions?: NestApplicationOptions);
-    selectContextModule(): void;
     applyOptions(): any;
     createServer(): any;
     setupModules(): Promise<void>;

lib/core/nest-application.js

@@ -52,10 +52,6 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
         this.config.setIoAdapter(ioAdapter);
         this.routesResolver = new routes_resolver_1.RoutesResolver(container, express_adapter_1.ExpressAdapter, this.config);
     }
-    selectContextModule() {
-        const modules = this.container.getModules().values();
-        this.contextModule = modules.next().value;
-    }
     applyOptions() {
         if (!this.appOptions) {
             return undefined;

lib/core/router/routes-resolver.d.ts

@@ -17,4 +17,5 @@ export declare class RoutesResolver implements Resolver {
     setupRouters(routes: Map<string, InstanceWrapper<Controller>>, moduleName: string, modulePath: string, express: Application): void;
     setupNotFoundHandler(express: Application): void;
     setupExceptionHandler(express: Application): void;
+    mapExternalException(err: any): any;
 }