Auto stash before merge of "master" and "origin/master"

Author: kamilmysliwiec

CHANGELOG.md

@@ -1,3 +1,11 @@
+## 4.6.1
+- **common**: [improvement] create `ModuleMetadata` interface
+- **common**: [bugfix] update `class-validator` #417
+- **core**: [feature] add `appOptions` property to `create[..]` methods of `NestFactory`
+- **core**: [improvement] majority of methods (`INestApplication`, `INestMicroservice`, and `INestApplicationContext`) return `this` instead of `void`
+- **core**: [refactor] remove static dependencies & relationships
+- **core**: [bugfix] catch error thrown by `body-parser` in exception filter #422
+
 ## 4.6.0
 - **common**: [feature] `ValidationPipe` improvements #388
 - **common**: [feature] `ParseIntPipe` improvements #85

lib/common/decorators/modules/module.decorator.d.ts

@@ -1,17 +1,12 @@
 import 'reflect-metadata';
+import { ModuleMetadata } from '../../interfaces/modules/module-metadata.interface';
 /**
  * Defines the module
- * - `modules` - @deprecated the set of the 'imported' modules
  * - `imports` - the set of the 'imported' modules
  * - `controllers` - the list of controllers (e.g. HTTP controllers)
  * - `components` - the list of components that belong to this module. They can be injected between themselves.
  * - `exports` - the set of components, which should be available for modules, which imports this module
+ * - `modules` - @deprecated the set of the 'imported' modules
  * @param obj {ModuleMetadata} Module metadata
  */
-export declare function Module(obj: {
-    modules?: any[];
-    imports?: any[];
-    controllers?: any[];
-    components?: any[];
-    exports?: any[];
-}): ClassDecorator;
+export declare function Module(obj: ModuleMetadata): ClassDecorator;

lib/common/decorators/modules/module.decorator.js

@@ -21,11 +21,11 @@ const validateKeys = (keys) => {
 };
 /**
  * Defines the module
- * - `modules` - @deprecated the set of the 'imported' modules
  * - `imports` - the set of the 'imported' modules
  * - `controllers` - the list of controllers (e.g. HTTP controllers)
  * - `components` - the list of components that belong to this module. They can be injected between themselves.
  * - `exports` - the set of components, which should be available for modules, which imports this module
+ * - `modules` - @deprecated the set of the 'imported' modules
  * @param obj {ModuleMetadata} Module metadata
  */
 function Module(obj) {

lib/common/interfaces/modules/module-metadata.interface.d.ts

@@ -1,9 +1,7 @@
-import { NestModule } from './nest-module.interface';
-import { Controller } from '../controllers/controller.interface';
 export interface ModuleMetadata {
-    modules?: NestModule[] | any[];
-    imports?: NestModule[] | any[];
+    imports?: any[];
     components?: any[];
-    controllers?: Controller[] | any[];
+    controllers?: any[];
     exports?: any[];
+    modules?: any[];
 }

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

@@ -9,48 +9,48 @@ export interface INestApplication extends INestApplicationContext {
      *
      * @returns Promise
      */
-    init(): Promise<void>;
+    init(): Promise<this>;
     /**
      * A wrapper function around native `express.use()` method.
      * Example `app.use(cors())`
      *
      * @returns void
      */
-    use(...args: any[]): void;
+    use(...args: any[]): this;
     /**
      * A wrapper function around native `express.set()` method.
      * Example `app.set('trust proxy', 'loopback')`
      *
      * @returns void
      */
-    set(...args: any[]): void;
+    set(...args: any[]): this;
     /**
      * A wrapper function around native `express.engine()` method.
      * Example `app.engine('mustache', mustacheExpress())`
      *
      * @returns void
      */
-    engine(...args: any[]): void;
+    engine(...args: any[]): this;
     /**
      * A wrapper function around native `express.enable()` method.
      * Example `app.enable('x-powered-by')`
      *
      * @returns void
      */
-    enable(...args: any[]): void;
+    enable(...args: any[]): this;
     /**
      * Enables CORS (Cross-Origin Resource Sharing)
      *
      * @returns void
      */
-    enableCors(): void;
+    enableCors(): this;
     /**
      * A wrapper function around native `express.disable()` method.
      * Example `app.disable('x-powered-by')`
      *
      * @returns void
      */
-    disable(...args: any[]): void;
+    disable(...args: any[]): this;
     /**
      * Starts the application.
      *
@@ -75,15 +75,15 @@ export interface INestApplication extends INestApplicationContext {
      * @param  {string} prefix The prefix for the every HTTP route path (for example `/v1/api`)
      * @returns void
      */
-    setGlobalPrefix(prefix: string): void;
+    setGlobalPrefix(prefix: string): this;
     /**
      * Setup Web Sockets Adapter, which will be used inside Gateways.
      * Use, when you want to override default `socket.io` library.
      *
      * @param  {WebSocketAdapter} adapter
      * @returns void
      */
-    useWebSocketAdapter(adapter: WebSocketAdapter): void;
+    useWebSocketAdapter(adapter: WebSocketAdapter): this;
     /**
      * Connects microservice to the NestApplication instance. It transforms application to the hybrid instance.
      *
@@ -109,7 +109,7 @@ export interface INestApplication extends INestApplicationContext {
      * @param  {Function} callback Optional callback function
      * @returns void
      */
-    startAllMicroservices(callback?: () => void): void;
+    startAllMicroservices(callback?: () => void): this;
     /**
      * Starts all the connected microservices and can be awaited
      *
@@ -121,25 +121,25 @@ export interface INestApplication extends INestApplicationContext {
      *
      * @param  {ExceptionFilter[]} ...filters
      */
-    useGlobalFilters(...filters: ExceptionFilter[]): any;
+    useGlobalFilters(...filters: ExceptionFilter[]): this;
     /**
      * Setups pipes as a global pipes (will be used within every HTTP route handler)
      *
      * @param  {PipeTransform[]} ...pipes
      */
-    useGlobalPipes(...pipes: PipeTransform<any>[]): any;
+    useGlobalPipes(...pipes: PipeTransform<any>[]): this;
     /**
      * Setups interceptors as a global interceptors (will be used within every HTTP route handler)
      *
      * @param  {NestInterceptor[]} ...interceptors
      */
-    useGlobalInterceptors(...interceptors: NestInterceptor[]): any;
+    useGlobalInterceptors(...interceptors: NestInterceptor[]): this;
     /**
      * Setups guards as a global guards (will be used within every HTTP route handler)
      *
      * @param  {CanActivate[]} ...guards
      */
-    useGlobalGuards(...guards: CanActivate[]): any;
+    useGlobalGuards(...guards: CanActivate[]): this;
     /**
      * Terminates the application (both NestApplication, Web Socket Gateways and every connected microservice)
      *

lib/common/interfaces/nest-microservice.interface.d.ts

@@ -25,31 +25,31 @@ export interface INestMicroservice extends INestApplicationContext {
      * @param  {WebSocketAdapter} adapter
      * @returns void
      */
-    useWebSocketAdapter(adapter: WebSocketAdapter): void;
+    useWebSocketAdapter(adapter: WebSocketAdapter): this;
     /**
      * Setups exception filters as a global filters (will be used within every message pattern handler)
      *
      * @param  {ExceptionFilter[]} ...filters
      */
-    useGlobalFilters(...filters: ExceptionFilter[]): any;
+    useGlobalFilters(...filters: ExceptionFilter[]): this;
     /**
      * Setups pipes as a global pipes (will be used within every message pattern handler)
      *
      * @param  {PipeTransform[]} ...pipes
      */
-    useGlobalPipes(...pipes: PipeTransform<any>[]): any;
+    useGlobalPipes(...pipes: PipeTransform<any>[]): this;
     /**
      * Setups interceptors as a global interceptors (will be used within every message pattern handler)
      *
      * @param  {NestInterceptor[]} ...interceptors
      */
-    useGlobalInterceptors(...interceptors: NestInterceptor[]): any;
+    useGlobalInterceptors(...interceptors: NestInterceptor[]): this;
     /**
      * Setups guards as a global guards (will be used within every message pattern handler)
      *
      * @param  {CanActivate[]} ...guards
      */
-    useGlobalGuards(...guards: CanActivate[]): any;
+    useGlobalGuards(...guards: CanActivate[]): this;
     /**
      * Terminates the application (both NestMicroservice and every Web Socket Gateway)
      *

lib/common/utils/forward-ref.util.d.ts

@@ -1,3 +1,3 @@
-export declare function forwardRef(fn: () => any): {
+export declare const forwardRef: (fn: () => any) => {
     forwardRef: () => any;
 };

lib/common/utils/forward-ref.util.js

@@ -1,6 +1,3 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-function forwardRef(fn) {
-    return { forwardRef: fn };
-}
-exports.forwardRef = forwardRef;
+exports.forwardRef = (fn) => ({ forwardRef: fn });

lib/common/utils/http-exception-body.util.d.ts

@@ -1,9 +1,8 @@
-export declare const createHttpExceptionBody: (message: any, error: string, status: number) => {
+export declare const createHttpExceptionBody: (message: any, error: string, statusCode: number) => {
     statusCode: number;
     error: string;
     message: any;
 } | {
     statusCode: number;
     error: string;
-    message?: undefined;
 };

lib/common/utils/http-exception-body.util.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.createHttpExceptionBody = (message, error, status) => message
-    ? { statusCode: status, error, message }
-    : { statusCode: status, error };
+exports.createHttpExceptionBody = (message, error, statusCode) => message
+    ? { statusCode, error, message }
+    : { statusCode, error };