@nestjs/core 6.11.4 → 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/discovery/discovery-module.d.ts

@@ -1,2 +1,5 @@
+/**
+ * @publicApi
+ */
 export declare class DiscoveryModule {
 }

package/discovery/discovery-module.js

@@ -4,6 +4,9 @@ const tslib_1 = require("tslib");
 const common_1 = require("@nestjs/common");
 const metadata_scanner_1 = require("../metadata-scanner");
 const discovery_service_1 = require("./discovery-service");
+/**
+ * @publicApi
+ */
 let DiscoveryModule = class DiscoveryModule {
 };
 DiscoveryModule = tslib_1.__decorate([

package/discovery/discovery-service.d.ts

@@ -1,9 +1,15 @@
 import { InstanceWrapper } from '../injector/instance-wrapper';
 import { Module } from '../injector/module';
 import { ModulesContainer } from '../injector/modules-container';
+/**
+ * @publicApi
+ */
 export interface DiscoveryOptions {
     include?: Function[];
 }
+/**
+ * @publicApi
+ */
 export declare class DiscoveryService {
     private readonly modulesContainer;
     constructor(modulesContainer: ModulesContainer);

package/discovery/discovery-service.js

@@ -3,6 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const tslib_1 = require("tslib");
 const common_1 = require("@nestjs/common");
 const modules_container_1 = require("../injector/modules-container");
+/**
+ * @publicApi
+ */
 let DiscoveryService = class DiscoveryService {
     constructor(modulesContainer) {
         this.modulesContainer = modulesContainer;

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/nest-application-context.d.ts

@@ -14,6 +14,7 @@ export declare class NestApplicationContext implements INestApplicationContext {
     private contextModule;
     protected isInitialized: boolean;
     protected readonly injector: Injector;
+    private shutdownCleanupRef?;
     private readonly activeShutdownSignals;
     private readonly containerScanner;
     constructor(container: NestContainer, scope?: Type<any>[], contextModule?: Module);
@@ -52,6 +53,10 @@ export declare class NestApplicationContext implements INestApplicationContext {
      * @param {string[]} signals The system signals it should listen to
      */
     protected listenToShutdownSignals(signals: string[]): void;
+    /**
+     * Unsubscribes from shutdown signals (process events)
+     */
+    protected unsubscribeFromProcessSignals(): void;
     /**
      * Calls the `onModuleInit` function on the registered
      * modules and its children.

package/nest-application-context.js

@@ -67,6 +67,7 @@ class NestApplicationContext {
         await this.callBeforeShutdownHook();
         await this.dispose();
         await this.callShutdownHook();
+        await this.unsubscribeFromProcessSignals();
     }
     useLogger(logger) {
         common_1.Logger.overrideLogger(logger);
@@ -125,11 +126,23 @@ class NestApplicationContext {
                 process.exit(1);
             }
         };
+        this.shutdownCleanupRef = cleanup;
         signals.forEach((signal) => {
             this.activeShutdownSignals.push(signal);
             process.on(signal, cleanup);
         });
     }
+    /**
+     * Unsubscribes from shutdown signals (process events)
+     */
+    unsubscribeFromProcessSignals() {
+        if (!this.shutdownCleanupRef) {
+            return;
+        }
+        this.activeShutdownSignals.forEach(signal => {
+            process.removeListener(signal, this.shutdownCleanupRef);
+        });
+    }
     /**
      * Calls the `onModuleInit` function on the registered
      * modules and its children.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "6.11.4",
+  "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.1",
+    "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);
             };

package/router/router-response-controller.d.ts

@@ -5,7 +5,7 @@ export interface CustomHeader {
 }
 export interface RedirectResponse {
     url: string;
-    statusCode: number;
+    statusCode?: number;
 }
 export declare class RouterResponseController {
     private readonly applicationRef;