@nestjs/core 6.11.7 → 6.11.11

package/Readme.md

@@ -9,7 +9,7 @@
     <p align="center">
 <a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/v/@nestjs/core.svg" alt="NPM Version" /></a>
 <a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/l/@nestjs/core.svg" alt="Package License" /></a>
-<a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/dm/@nestjs/core.svg" alt="NPM Downloads" /></a>
+<a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/dm/@nestjs/common.svg" alt="NPM Downloads" /></a>
 <a href="https://circleci.com/gh/nestjs/nest" target="_blank"><img src="https://img.shields.io/circleci/build/github/nestjs/nest/master" alt="CircleCI" /></a>
 <a href="https://coveralls.io/github/nestjs/nest?branch=master" target="_blank"><img src="https://coveralls.io/repos/github/nestjs/nest/badge.svg?branch=master#9" alt="Coverage" /></a>
 <a href="https://discord.gg/G7Qnnhy" target="_blank"><img src="https://img.shields.io/badge/discord-online-brightgreen.svg" alt="Discord"/></a>

package/helpers/http-adapter-host.d.ts

@@ -18,11 +18,12 @@ export declare class HttpAdapterHost<T extends AbstractHttpAdapter = AbstractHtt
      *
      * @param httpAdapter reference to the `HttpAdapter` to be set
      */
+    set httpAdapter(httpAdapter: T);
     /**
-    * Accessor for the underlying `HttpAdapter`
-    *
-    * @example
-    * `const httpAdapter = adapterHost.httpAdapter;`
-    */
-    httpAdapter: T;
+     * Accessor for the underlying `HttpAdapter`
+     *
+     * @example
+     * `const httpAdapter = adapterHost.httpAdapter;`
+     */
+    get httpAdapter(): T;
 }

package/hooks/before-app-shutdown.hook.js

@@ -30,14 +30,16 @@ function callOperator(instances, signal) {
  */
 async function callBeforeAppShutdownHook(module, signal) {
     const providers = module.getNonAliasProviders();
-    const [_, { instance: moduleClassInstance }] = providers.shift();
+    const [_, moduleClassHost] = providers.shift();
     const instances = [...module.controllers, ...providers];
     const nonTransientInstances = transient_instances_1.getNonTransientInstances(instances);
     await Promise.all(callOperator(nonTransientInstances, signal));
     const transientInstances = transient_instances_1.getTransientInstances(instances);
     await Promise.all(callOperator(transientInstances, signal));
+    const moduleClassInstance = moduleClassHost.instance;
     if (moduleClassInstance &&
-        hasBeforeApplicationShutdownHook(moduleClassInstance)) {
+        hasBeforeApplicationShutdownHook(moduleClassInstance) &&
+        moduleClassHost.isDependencyTreeStatic()) {
         await moduleClassInstance.beforeApplicationShutdown(signal);
     }
 }

package/hooks/on-app-bootstrap.hook.js

@@ -31,7 +31,7 @@ async function callModuleBootstrapHook(module) {
     const providers = module.getNonAliasProviders();
     // Module (class) instance is the first element of the providers array
     // Lifecycle hook has to be called once all classes are properly initialized
-    const [_, { instance: moduleClassInstance }] = providers.shift();
+    const [_, moduleClassHost] = providers.shift();
     const instances = [
         ...module.controllers,
         ...providers,
@@ -43,7 +43,10 @@ async function callModuleBootstrapHook(module) {
     const transientInstances = transient_instances_1.getTransientInstances(instances);
     await Promise.all(callOperator(transientInstances));
     // Call the instance itself
-    if (moduleClassInstance && hasOnAppBootstrapHook(moduleClassInstance)) {
+    const moduleClassInstance = moduleClassHost.instance;
+    if (moduleClassInstance &&
+        hasOnAppBootstrapHook(moduleClassInstance) &&
+        moduleClassHost.isDependencyTreeStatic()) {
         await moduleClassInstance.onApplicationBootstrap();
     }
 }

package/hooks/on-app-shutdown.hook.js

@@ -31,7 +31,7 @@ async function callAppShutdownHook(module, signal) {
     const providers = module.getNonAliasProviders();
     // Module (class) instance is the first element of the providers array
     // Lifecycle hook has to be called once all classes are properly initialized
-    const [_, { instance: moduleClassInstance }] = providers.shift();
+    const [_, moduleClassHost] = providers.shift();
     const instances = [
         ...module.controllers,
         ...providers,
@@ -43,7 +43,10 @@ async function callAppShutdownHook(module, signal) {
     const transientInstances = transient_instances_1.getTransientInstances(instances);
     await Promise.all(callOperator(transientInstances, signal));
     // Call the instance itself
-    if (moduleClassInstance && hasOnAppShutdownHook(moduleClassInstance)) {
+    const moduleClassInstance = moduleClassHost.instance;
+    if (moduleClassInstance &&
+        hasOnAppShutdownHook(moduleClassInstance) &&
+        moduleClassHost.isDependencyTreeStatic()) {
         await moduleClassInstance.onApplicationShutdown(signal);
     }
 }

package/hooks/on-module-destroy.hook.js

@@ -31,7 +31,7 @@ async function callModuleDestroyHook(module) {
     const providers = module.getNonAliasProviders();
     // Module (class) instance is the first element of the providers array
     // Lifecycle hook has to be called once all classes are properly destroyed
-    const [_, { instance: moduleClassInstance }] = providers.shift();
+    const [_, moduleClassHost] = providers.shift();
     const instances = [
         ...module.controllers,
         ...providers,
@@ -43,7 +43,10 @@ async function callModuleDestroyHook(module) {
     const transientInstances = transient_instances_1.getTransientInstances(instances);
     await Promise.all(callOperator(transientInstances));
     // Call the module instance itself
-    if (moduleClassInstance && hasOnModuleDestroyHook(moduleClassInstance)) {
+    const moduleClassInstance = moduleClassHost.instance;
+    if (moduleClassInstance &&
+        hasOnModuleDestroyHook(moduleClassInstance) &&
+        moduleClassHost.isDependencyTreeStatic()) {
         await moduleClassInstance.onModuleDestroy();
     }
 }

package/hooks/on-module-init.hook.js

@@ -31,7 +31,7 @@ async function callModuleInitHook(module) {
     const providers = module.getNonAliasProviders();
     // Module (class) instance is the first element of the providers array
     // Lifecycle hook has to be called once all classes are properly initialized
-    const [_, { instance: moduleClassInstance }] = providers.shift();
+    const [_, moduleClassHost] = providers.shift();
     const instances = [
         ...module.controllers,
         ...providers,
@@ -43,7 +43,10 @@ async function callModuleInitHook(module) {
     const transientInstances = transient_instances_1.getTransientInstances(instances);
     await Promise.all(callOperator(transientInstances));
     // Call the instance itself
-    if (moduleClassInstance && hasOnModuleInitHook(moduleClassInstance)) {
+    const moduleClassInstance = moduleClassHost.instance;
+    if (moduleClassInstance &&
+        hasOnModuleInitHook(moduleClassInstance) &&
+        moduleClassHost.isDependencyTreeStatic()) {
         await moduleClassInstance.onModuleInit();
     }
 }

package/injector/container.d.ts

@@ -16,7 +16,7 @@ export declare class NestContainer {
     private readonly internalProvidersStorage;
     private internalCoreModule;
     constructor(_applicationConfig?: ApplicationConfig);
-    readonly applicationConfig: ApplicationConfig | undefined;
+    get applicationConfig(): ApplicationConfig | undefined;
     setHttpAdapter(httpAdapter: any): void;
     getHttpAdapterRef(): import("..").AbstractHttpAdapter<any, any, any>;
     addModule(metatype: Type<any> | DynamicModule | Promise<DynamicModule>, scope: Type<any>[]): Promise<Module>;

package/injector/instance-wrapper.d.ts

@@ -30,10 +30,11 @@ export declare class InstanceWrapper<T = any> {
     private transientMap?;
     private isTreeStatic;
     constructor(metadata?: Partial<InstanceWrapper<T>> & Partial<InstancePerContext<T>>);
-    readonly id: string;
-    instance: T;
-    readonly isNotMetatype: boolean;
-    readonly isTransient: boolean;
+    get id(): string;
+    set instance(value: T);
+    get instance(): T;
+    get isNotMetatype(): boolean;
+    get isTransient(): boolean;
     getInstanceByContextId(contextId: ContextId, inquirerId?: string): InstancePerContext<T>;
     getInstanceByInquirerId(contextId: ContextId, inquirerId: string): InstancePerContext<T>;
     setInstanceByContextId(contextId: ContextId, value: InstancePerContext<T>, inquirerId?: string): void;

package/injector/internal-providers-storage.d.ts

@@ -3,6 +3,7 @@ import { AbstractHttpAdapter } from '../adapters';
 export declare class InternalProvidersStorage {
     private readonly _httpAdapterHost;
     private _httpAdapter;
-    readonly httpAdapterHost: HttpAdapterHost;
-    httpAdapter: AbstractHttpAdapter;
+    get httpAdapterHost(): HttpAdapterHost;
+    get httpAdapter(): AbstractHttpAdapter;
+    set httpAdapter(httpAdapter: AbstractHttpAdapter);
 }

package/injector/module.d.ts

@@ -18,29 +18,30 @@ export declare class Module {
     private readonly _exports;
     private _distance;
     constructor(_metatype: Type<any>, _scope: Type<any>[], container: NestContainer);
-    readonly id: string;
-    readonly scope: Type<any>[];
-    readonly providers: Map<any, InstanceWrapper<Injectable>>;
-    readonly middlewares: Map<any, InstanceWrapper<Injectable>>;
-    readonly imports: Set<Module>;
+    get id(): string;
+    get scope(): Type<any>[];
+    get providers(): Map<any, InstanceWrapper<Injectable>>;
+    get middlewares(): Map<any, InstanceWrapper<Injectable>>;
+    get imports(): Set<Module>;
     /**
      * Left for backward-compatibility reasons
      */
-    readonly relatedModules: Set<Module>;
+    get relatedModules(): Set<Module>;
     /**
      * Left for backward-compatibility reasons
      */
-    readonly components: Map<string, InstanceWrapper<Injectable>>;
+    get components(): Map<string, InstanceWrapper<Injectable>>;
     /**
      * Left for backward-compatibility reasons
      */
-    readonly routes: Map<string, InstanceWrapper<Controller>>;
-    readonly injectables: Map<string, InstanceWrapper<Injectable>>;
-    readonly controllers: Map<string, InstanceWrapper<Controller>>;
-    readonly exports: Set<string | symbol>;
-    readonly instance: NestModule;
-    readonly metatype: Type<any>;
-    distance: number;
+    get routes(): Map<string, InstanceWrapper<Controller>>;
+    get injectables(): Map<string, InstanceWrapper<Injectable>>;
+    get controllers(): Map<string, InstanceWrapper<Controller>>;
+    get exports(): Set<string | symbol>;
+    get instance(): NestModule;
+    get metatype(): Type<any>;
+    get distance(): number;
+    set distance(value: number);
     addCoreProviders(): void;
     addModuleRef(): void;
     addModuleAsProvider(): void;

package/middleware/utils.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
-const uuid = require("uuid/v4");
+const uuid_1 = require("uuid");
 exports.filterMiddleware = (middleware) => {
     return []
         .concat(middleware)
@@ -23,7 +23,7 @@ function isClass(middleware) {
 }
 exports.isClass = isClass;
 function assignToken(metatype) {
-    Object.defineProperty(metatype, 'name', { value: uuid() });
+    Object.defineProperty(metatype, 'name', { value: uuid_1.v4() });
     return metatype;
 }
 exports.assignToken = assignToken;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "6.11.7",
+  "version": "6.11.11",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -30,10 +30,10 @@
     "@nuxtjs/opencollective": "0.2.2",
     "fast-safe-stringify": "2.0.7",
     "iterare": "1.2.0",
-    "object-hash": "2.0.2",
+    "object-hash": "2.0.3",
     "path-to-regexp": "3.2.0",
-    "tslib": "1.10.0",
-    "uuid": "3.4.0"
+    "tslib": "1.11.1",
+    "uuid": "7.0.1"
   },
   "peerDependencies": {
     "@nestjs/common": "^6.0.0",

package/router/router-execution-context.js

@@ -156,7 +156,7 @@ class RouterExecutionContext {
                 await this.responseController.render(result, res, renderTemplate);
             };
         }
-        if (redirectResponse && redirectResponse.url) {
+        if (redirectResponse && typeof redirectResponse.url === 'string') {
             return async (result, res) => {
                 await this.responseController.redirect(result, res, redirectResponse);
             };