@nestjs/core 6.11.7 → 6.11.8

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/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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "6.11.7",
+  "version": "6.11.8",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -30,7 +30,7 @@
     "@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"

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);
             };