@nestjs/core 8.2.0 → 8.2.4

package/Readme.md

@@ -57,10 +57,13 @@ With official support, you can get expert help straight from Nest core team. We
 Nest is an MIT-licensed open source project. It can grow thanks to the sponsors and support by the amazing backers. If you'd like to join them, please [read more here](https://docs.nestjs.com/support).
 
 #### Principal Sponsors
-<table style="text-align:center;"><tr><td>
+<table style="text-align:center;"><tr>
+  <td><a href="https://trilon.io" target="_blank"><img src="https://nestjs.com/img/trilon.svg" width="200" valign="middle" /></a></td>
+  <td><a href="https://www.redhat.com" target="_blank"><img src="https://nestjs.com/img/red-hat-logo.svg" width="200" valign="middle" /></a></td>
+<td>
 <a href="https://github.com/Sanofi-IADC" target="_blank"><img src="https://docs.nestjs.com/assets/sponsors/sanofi.png" width="180" valign="middle" /></a></td>
 <td>
-<a href="https://trilon.io" target="_blank"><img src="https://nestjs.com/img/trilon.svg" width="200" valign="middle" /></a></td>
+<a href="https://nx.dev" target="_blank"><img src="https://nestjs.com/img/nx-logo.png" height="45" valign="middle" /></a></td>
 </tr></table>
 
 #### Gold Sponsors
@@ -78,7 +81,7 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
 <table style="text-align:center;"><tr><td>
 <a href="https://neoteric.eu/" target="_blank"><img src="https://nestjs.com/img/neoteric-cut.png" width="120" valign="middle" /></a> </td><td>
   <a href="http://gojob.com" target="_blank"><img src="http://nestjs.com/img/gojob-logo.png" valign="middle" width="100" /></a> </td><td>
-<a href="http://www.leogistics.com" target="_blank"><img src="https://nestjs.com/img/leogistics-logo.jpeg" width="150" valign="middle" /></td><td>
+<a href="https://www.myleodsc.com/" target="_blank"><img src="https://nestjs.com/img/myleo-logo.png" width="180" valign="middle" /></td><td>
 <a href="https://careers.meetdandy.com/?gh_src=063ba61e3us" target="_blank"><img src="https://nestjs.com/img/dandy-roles-logo.svg" width="150" valign="middle" /></td><td>
 <a href="https://www.castlecraft.in" target="_blank"><img src="https://nestjs.com/img/castlecraft-logo.png" width="150" valign="middle" /></td>
 <td><a href="https://www.tinystacks.com" target="_blank"><img src="https://nestjs.com/img/tinystacks-logo.png#1" width="140" valign="middle" /></td></tr></table>
@@ -113,6 +116,8 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
   <td align="center" valign="middle"><a href="https://studyclerk.com/pay-for-research-paper" target="_blank"><img src="https://nestjs.com/img/studyclerk-logo.png" width="125" valign="middle" /></a></td><td align="center" valign="middle"><a href="https://xyndata.com" target="_blank"><img src="https://nestjs.com/img/xyndata-logo.png" width="125" valign="middle" /></a></td></tr><tr>
   <td align="center" valign="middle"><a href="https://www.anonymistic.com/" target="_blank"><img src="https://nestjs.com/img/anonymistic-logo.png" width="125" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://www.naologic.com/" target="_blank"><img src="https://nestjs.com/img/naologic-logo.svg" width="125" valign="middle" /></a></td>
+  <td align="center" valign="middle"><a href="https://messaged.com/" target="_blank"><img src="https://nestjs.com/img/messaged-logo.png" width="50" valign="middle" /></a></td>
+  <td align="center" valign="middle"><a href="https://triplecore.io" target="_blank"><img src="https://nestjs.com/img/triplecore-logo.svg" width="50" valign="middle" /></a></td>
   </tr></table>
 
 ## Backers

package/application-config.js

@@ -94,6 +94,10 @@ class ApplicationConfig {
         return this.globalRequestGuards;
     }
     enableVersioning(options) {
+        if (Array.isArray(options.defaultVersion)) {
+            // Drop duplicated versions
+            options.defaultVersion = Array.from(new Set(options.defaultVersion));
+        }
         this.versioningOptions = options;
     }
     getVersioning() {

package/errors/exceptions/invalid-class-module.exception.d.ts

@@ -0,0 +1,4 @@
+import { RuntimeException } from './runtime.exception';
+export declare class InvalidClassModuleException extends RuntimeException {
+    constructor(metatypeUsedAsAModule: any, scope: any[]);
+}

package/errors/exceptions/invalid-class-module.exception.js

@@ -0,0 +1,11 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.InvalidClassModuleException = void 0;
+const messages_1 = require("../messages");
+const runtime_exception_1 = require("./runtime.exception");
+class InvalidClassModuleException extends runtime_exception_1.RuntimeException {
+    constructor(metatypeUsedAsAModule, scope) {
+        super(messages_1.USING_INVALID_CLASS_AS_A_MODULE_MESSAGE(metatypeUsedAsAModule, scope));
+    }
+}
+exports.InvalidClassModuleException = InvalidClassModuleException;

package/errors/messages.d.ts

@@ -1,10 +1,11 @@
-import { Type } from '@nestjs/common';
+import type { ForwardReference, Type } from '@nestjs/common';
 import { InjectorDependencyContext } from '../injector/injector';
 import { Module } from '../injector/module';
 export declare const UNKNOWN_DEPENDENCIES_MESSAGE: (type: string | symbol, unknownDependencyContext: InjectorDependencyContext, module: Module) => string;
 export declare const INVALID_MIDDLEWARE_MESSAGE: (text: TemplateStringsArray, name: string) => string;
 export declare const UNDEFINED_FORWARDREF_MESSAGE: (scope: Type<any>[]) => string;
 export declare const INVALID_MODULE_MESSAGE: (parentModule: any, index: number, scope: any[]) => string;
+export declare const USING_INVALID_CLASS_AS_A_MODULE_MESSAGE: (metatypeUsedAsAModule: Type | ForwardReference, scope: any[]) => string;
 export declare const UNDEFINED_MODULE_MESSAGE: (parentModule: any, index: number, scope: any[]) => string;
 export declare const UNKNOWN_EXPORT_MESSAGE: (token: string | symbol, module: string) => string;
 export declare const INVALID_CLASS_MESSAGE: (text: TemplateStringsArray, value: any) => string;

package/errors/messages.js

@@ -1,17 +1,20 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.MICROSERVICES_PACKAGE_NOT_FOUND_EXCEPTION = exports.INVALID_EXCEPTION_FILTER = exports.UNHANDLED_RUNTIME_EXCEPTION = exports.UNKNOWN_REQUEST_MAPPING = exports.INVALID_MIDDLEWARE_CONFIGURATION = exports.INVALID_CLASS_SCOPE_MESSAGE = exports.INVALID_CLASS_MESSAGE = exports.UNKNOWN_EXPORT_MESSAGE = exports.UNDEFINED_MODULE_MESSAGE = exports.INVALID_MODULE_MESSAGE = exports.UNDEFINED_FORWARDREF_MESSAGE = exports.INVALID_MIDDLEWARE_MESSAGE = exports.UNKNOWN_DEPENDENCIES_MESSAGE = void 0;
+exports.MICROSERVICES_PACKAGE_NOT_FOUND_EXCEPTION = exports.INVALID_EXCEPTION_FILTER = exports.UNHANDLED_RUNTIME_EXCEPTION = exports.UNKNOWN_REQUEST_MAPPING = exports.INVALID_MIDDLEWARE_CONFIGURATION = exports.INVALID_CLASS_SCOPE_MESSAGE = exports.INVALID_CLASS_MESSAGE = exports.UNKNOWN_EXPORT_MESSAGE = exports.UNDEFINED_MODULE_MESSAGE = exports.USING_INVALID_CLASS_AS_A_MODULE_MESSAGE = exports.INVALID_MODULE_MESSAGE = exports.UNDEFINED_FORWARDREF_MESSAGE = exports.INVALID_MIDDLEWARE_MESSAGE = exports.UNKNOWN_DEPENDENCIES_MESSAGE = void 0;
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
 /**
- * Returns the name of an instance
+ * Returns the name of an instance or `undefined`
  * @param instance The instance which should get the name from
  */
 const getInstanceName = (instance) => {
-    var _a, _b, _c;
+    var _a, _b, _c, _d, _e;
     if ((_a = instance) === null || _a === void 0 ? void 0 : _a.forwardRef) {
         return (_b = instance.forwardRef()) === null || _b === void 0 ? void 0 : _b.name;
     }
-    return (_c = instance) === null || _c === void 0 ? void 0 : _c.name;
+    if ((_c = instance) === null || _c === void 0 ? void 0 : _c.module) {
+        return (_d = instance.module) === null || _d === void 0 ? void 0 : _d.name;
+    }
+    return (_e = instance) === null || _e === void 0 ? void 0 : _e.name;
 };
 /**
  * Returns the name of the dependency
@@ -67,16 +70,26 @@ const UNDEFINED_FORWARDREF_MESSAGE = (scope) => `Nest cannot create the module i
 
 (Read more: https://docs.nestjs.com/fundamentals/circular-dependency)
 Scope [${stringifyScope(scope)}]
-  `;
+`;
 exports.UNDEFINED_FORWARDREF_MESSAGE = UNDEFINED_FORWARDREF_MESSAGE;
 const INVALID_MODULE_MESSAGE = (parentModule, index, scope) => {
     const parentModuleName = (parentModule === null || parentModule === void 0 ? void 0 : parentModule.name) || 'module';
     return `Nest cannot create the ${parentModuleName} instance.
-Received an unexpected value at index [${index}] of the ${parentModuleName} "imports" array. 
+Received an unexpected value at index [${index}] of the ${parentModuleName} "imports" array.
 
 Scope [${stringifyScope(scope)}]`;
 };
 exports.INVALID_MODULE_MESSAGE = INVALID_MODULE_MESSAGE;
+const USING_INVALID_CLASS_AS_A_MODULE_MESSAGE = (metatypeUsedAsAModule, scope) => {
+    const metatypeName = getInstanceName(metatypeUsedAsAModule) || 'found';
+    // TODO(v9): Edit the message below:
+    return `In the next major version, Nest will not allow classes annotated with @Injectable(), @Catch(), and @Controller() decorators to appear in the "imports" array of a module.
+Please remove "${metatypeName}" (including forwarded occurrences, if any) from all of the "imports" arrays.
+
+Scope [${stringifyScope(scope)}]
+`;
+};
+exports.USING_INVALID_CLASS_AS_A_MODULE_MESSAGE = USING_INVALID_CLASS_AS_A_MODULE_MESSAGE;
 const UNDEFINED_MODULE_MESSAGE = (parentModule, index, scope) => {
     const parentModuleName = (parentModule === null || parentModule === void 0 ? void 0 : parentModule.name) || 'module';
     return `Nest cannot create the ${parentModuleName} instance.

package/helpers/messages.js

@@ -8,19 +8,21 @@ exports.MODULE_INIT_MESSAGE = MODULE_INIT_MESSAGE;
 const ROUTE_MAPPED_MESSAGE = (path, method) => `Mapped {${path}, ${request_method_enum_1.RequestMethod[method]}} route`;
 exports.ROUTE_MAPPED_MESSAGE = ROUTE_MAPPED_MESSAGE;
 const VERSIONED_ROUTE_MAPPED_MESSAGE = (path, method, version) => {
-    if (version === version_options_interface_1.VERSION_NEUTRAL) {
-        version = 'Neutral';
-    }
-    return `Mapped {${path}, ${request_method_enum_1.RequestMethod[method]}} (version: ${version}) route`;
+    const controllerVersions = Array.isArray(version) ? version : [version];
+    const versions = controllerVersions
+        .map(version => (version === version_options_interface_1.VERSION_NEUTRAL ? 'Neutral' : version))
+        .join(',');
+    return `Mapped {${path}, ${request_method_enum_1.RequestMethod[method]}} (version: ${versions}) route`;
 };
 exports.VERSIONED_ROUTE_MAPPED_MESSAGE = VERSIONED_ROUTE_MAPPED_MESSAGE;
 const CONTROLLER_MAPPING_MESSAGE = (name, path) => `${name} {${path}}:`;
 exports.CONTROLLER_MAPPING_MESSAGE = CONTROLLER_MAPPING_MESSAGE;
 const VERSIONED_CONTROLLER_MAPPING_MESSAGE = (name, path, version) => {
-    if (version === version_options_interface_1.VERSION_NEUTRAL) {
-        version = 'Neutral';
-    }
-    return `${name} {${path}} (version: ${version}):`;
+    const controllerVersions = Array.isArray(version) ? version : [version];
+    const versions = controllerVersions
+        .map(version => (version === version_options_interface_1.VERSION_NEUTRAL ? 'Neutral' : version))
+        .join(',');
+    return `${name} {${path}} (version: ${versions}):`;
 };
 exports.VERSIONED_CONTROLLER_MAPPING_MESSAGE = VERSIONED_CONTROLLER_MAPPING_MESSAGE;
 const INVALID_EXECUTION_CONTEXT = (methodName, currentContext) => `Calling ${methodName} is not allowed in this context. Your current execution context is "${currentContext}".`;

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

@@ -10,7 +10,7 @@ const transient_instances_1 = require("../injector/helpers/transient-instances")
  * @param instance The instance which should be checked
  */
 function hasBeforeApplicationShutdownHook(instance) {
-    return !shared_utils_1.isNil(instance.beforeApplicationShutdown);
+    return shared_utils_1.isFunction(instance.beforeApplicationShutdown);
 }
 /**
  * Calls the given instances

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

@@ -10,7 +10,7 @@ const transient_instances_1 = require("../injector/helpers/transient-instances")
  * @param instance The instance which should be checked
  */
 function hasOnAppBootstrapHook(instance) {
-    return !shared_utils_1.isNil(instance.onApplicationBootstrap);
+    return shared_utils_1.isFunction(instance.onApplicationBootstrap);
 }
 /**
  * Calls the given instances

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

@@ -10,7 +10,7 @@ const transient_instances_1 = require("../injector/helpers/transient-instances")
  * @param instance The instance which should be checked
  */
 function hasOnAppShutdownHook(instance) {
-    return !shared_utils_1.isNil(instance.onApplicationShutdown);
+    return shared_utils_1.isFunction(instance.onApplicationShutdown);
 }
 /**
  * Calls the given instances

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

@@ -10,7 +10,7 @@ const transient_instances_1 = require("../injector/helpers/transient-instances")
  * @param instance The instance which should be checked
  */
 function hasOnModuleDestroyHook(instance) {
-    return !shared_utils_1.isNil(instance.onModuleDestroy);
+    return shared_utils_1.isFunction(instance.onModuleDestroy);
 }
 /**
  * Calls the given instances onModuleDestroy hook

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

@@ -10,7 +10,7 @@ const transient_instances_1 = require("../injector/helpers/transient-instances")
  * @param instance The instance which should be checked
  */
 function hasOnModuleInitHook(instance) {
-    return !shared_utils_1.isNil(instance.onModuleInit);
+    return shared_utils_1.isFunction(instance.onModuleInit);
 }
 /**
  * Calls the given instances

package/injector/injector.d.ts

@@ -71,7 +71,7 @@ export declare class Injector {
     private resolveScopedComponentHost;
     private isInquirerRequest;
     private isInquirer;
-    private addDependencyMetadata;
+    protected addDependencyMetadata(keyOrIndex: number | string, hostWrapper: InstanceWrapper, instanceWrapper: InstanceWrapper): void;
     private getTokenName;
     private printResolvingDependenciesLog;
     private printLookingForProviderLog;

package/injector/injector.js

@@ -137,6 +137,7 @@ class Injector {
     }
     async resolveSingleParam(wrapper, param, dependencyContext, moduleRef, contextId = constants_2.STATIC_CONTEXT, inquirer, keyOrIndex) {
         if (shared_utils_1.isUndefined(param)) {
+            this.logger.log('Nest encountered an undefined dependency. This may be due to a circular import or a missing dependency declaration.');
             throw new undefined_dependency_exception_1.UndefinedDependencyException(wrapper.name, dependencyContext, moduleRef);
         }
         const token = this.resolveParamToken(wrapper, param);
@@ -364,12 +365,12 @@ class Injector {
         return shared_utils_1.isFunction(token) ? token.name : token.toString();
     }
     printResolvingDependenciesLog(token, inquirer) {
-        var _a;
+        var _a, _b, _c;
         if (!this.isDebugMode()) {
             return;
         }
         const tokenName = this.getTokenName(token);
-        const dependentName = (_a = inquirer === null || inquirer === void 0 ? void 0 : inquirer.name) !== null && _a !== void 0 ? _a : 'unknown';
+        const dependentName = (_c = ((inquirer === null || inquirer === void 0 ? void 0 : inquirer.name) && ((_b = (_a = inquirer.name).toString) === null || _b === void 0 ? void 0 : _b.call(_a)))) !== null && _c !== void 0 ? _c : 'unknown';
         const isAlias = dependentName === tokenName;
         const messageToPrint = `Resolving dependency ${cli_colors_util_1.clc.cyanBright(tokenName)}${cli_colors_util_1.clc.green(' in the ')}${cli_colors_util_1.clc.yellow(dependentName)}${cli_colors_util_1.clc.green(` provider ${isAlias ? '(alias)' : ''}`)}`;
         this.logger.log(messageToPrint);

package/injector/instance-loader.d.ts

@@ -1,10 +1,11 @@
 import { Logger } from '@nestjs/common';
 import { NestContainer } from './container';
+import { Injector } from './injector';
 import { Module } from './module';
 export declare class InstanceLoader {
-    private readonly container;
+    protected readonly container: NestContainer;
     private readonly logger;
-    private readonly injector;
+    protected readonly injector: Injector;
     constructor(container: NestContainer, logger?: Logger);
     createInstancesOfDependencies(modules?: Map<string, Module>): Promise<void>;
     private createPrototypes;

package/injector/module-ref.d.ts

@@ -8,10 +8,10 @@ export declare abstract class ModuleRef {
     private _instanceLinksHost;
     private get instanceLinksHost();
     constructor(container: NestContainer);
-    abstract get<TInput = any, TResult = TInput>(typeOrToken: Type<TInput> | string | symbol, options?: {
+    abstract get<TInput = any, TResult = TInput>(typeOrToken: Type<TInput> | Function | string | symbol, options?: {
         strict: boolean;
     }): TResult;
-    abstract resolve<TInput = any, TResult = TInput>(typeOrToken: Type<TInput> | string | symbol, contextId?: ContextId, options?: {
+    abstract resolve<TInput = any, TResult = TInput>(typeOrToken: Type<TInput> | Function | string | symbol, contextId?: ContextId, options?: {
         strict: boolean;
     }): Promise<TResult>;
     abstract create<T = any>(type: Type<T>): Promise<T>;

package/injector/module-token-factory.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ModuleTokenFactory = void 0;
 const random_string_generator_util_1 = require("@nestjs/common/utils/random-string-generator.util");
+const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
 const fast_safe_stringify_1 = require("fast-safe-stringify");
 const hash = require("object-hash");
 class ModuleTokenFactory {
@@ -38,7 +39,7 @@ class ModuleTokenFactory {
         return metatype.name;
     }
     replacer(key, value) {
-        if (typeof value === 'function') {
+        if (shared_utils_1.isFunction(value)) {
             const funcAsString = value.toString();
             const isClass = /^class\s/.test(funcAsString);
             if (isClass) {
@@ -46,7 +47,7 @@ class ModuleTokenFactory {
             }
             return hash(funcAsString, { ignoreUnknown: true });
         }
-        if (typeof value === 'symbol') {
+        if (shared_utils_1.isSymbol(value)) {
             return value.toString();
         }
         return value;

package/middleware/utils.js

@@ -63,7 +63,7 @@ function isMiddlewareClass(middleware) {
     const middlewareArr = middlewareStr.split(' ');
     return (middlewareArr[0] === 'function' &&
         /[A-Z]/.test((_a = middlewareArr[1]) === null || _a === void 0 ? void 0 : _a[0]) &&
-        typeof ((_b = middleware.prototype) === null || _b === void 0 ? void 0 : _b.use) === 'function');
+        shared_utils_1.isFunction((_b = middleware.prototype) === null || _b === void 0 ? void 0 : _b.use));
 }
 exports.isMiddlewareClass = isMiddlewareClass;
 function assignToken(metatype, token = uuid_1.v4()) {

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 shouldFlushLogsOnOverride;
     private readonly activeShutdownSignals;
     private readonly moduleCompiler;
     private shutdownCleanupRef?;
@@ -40,6 +41,10 @@ export declare class NestApplicationContext implements INestApplicationContext {
     close(): Promise<void>;
     useLogger(logger: LoggerService | LogLevel[] | false): void;
     flushLogs(): void;
+    /**
+     * Define that it must flush logs right after defining a custom logger.
+     */
+    flushLogsOnOverride(): void;
     /**
      * Enables the usage of shutdown hooks. Will call the
      * `onApplicationShutdown` function of a provider if the

package/nest-application-context.js

@@ -24,6 +24,7 @@ class NestApplicationContext {
         this.contextModule = contextModule;
         this.isInitialized = false;
         this.injector = new injector_1.Injector();
+        this.shouldFlushLogsOnOverride = false;
         this.activeShutdownSignals = new Array();
         this.moduleCompiler = new compiler_1.ModuleCompiler();
     }
@@ -85,10 +86,19 @@ class NestApplicationContext {
     }
     useLogger(logger) {
         common_1.Logger.overrideLogger(logger);
+        if (this.shouldFlushLogsOnOverride) {
+            this.flushLogs();
+        }
     }
     flushLogs() {
         common_1.Logger.flush();
     }
+    /**
+     * Define that it must flush logs right after defining a custom logger.
+     */
+    flushLogsOnOverride() {
+        this.shouldFlushLogsOnOverride = true;
+    }
     /**
      * Enables the usage of shutdown hooks. Will call the
      * `onApplicationShutdown` function of a provider if the

package/nest-application.js

@@ -61,8 +61,7 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
         if (!this.appOptions || !this.appOptions.cors) {
             return undefined;
         }
-        const passCustomOptions = shared_utils_1.isObject(this.appOptions.cors) ||
-            typeof this.appOptions.cors === 'function';
+        const passCustomOptions = shared_utils_1.isObject(this.appOptions.cors) || shared_utils_1.isFunction(this.appOptions.cors);
         if (!passCustomOptions) {
             return this.enableCors();
         }
@@ -200,7 +199,7 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
         });
     }
     formatAddress(address) {
-        if (typeof address === 'string') {
+        if (shared_utils_1.isString(address)) {
             if (os_1.platform() === 'win32') {
                 return address;
             }
@@ -276,7 +275,7 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
     }
     host() {
         const address = this.httpServer.address();
-        if (typeof address === 'string') {
+        if (shared_utils_1.isString(address)) {
             return undefined;
         }
         return address && address.address;

package/nest-factory.js

@@ -74,6 +74,9 @@ class NestFactoryStatic {
         const modules = container.getModules().values();
         const root = modules.next().value;
         const context = this.createNestInstance(new nest_application_context_1.NestApplicationContext(container, [], root));
+        if (this.autoFlushLogs) {
+            context.flushLogsOnOverride();
+        }
         return context.init();
     }
     createNestInstance(instance) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "8.2.0",
+  "version": "8.2.4",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -36,7 +36,7 @@
     "uuid": "8.3.2"
   },
   "devDependencies": {
-    "@nestjs/common": "8.2.0"
+    "@nestjs/common": "8.2.4"
   },
   "peerDependencies": {
     "@nestjs/common": "^8.0.0",

package/router/route-path-factory.js

@@ -11,16 +11,21 @@ class RoutePathFactory {
     create(metadata, requestMethod) {
         var _a;
         let paths = [''];
-        const version = this.getVersion(metadata);
-        if (version && ((_a = metadata.versioningOptions) === null || _a === void 0 ? void 0 : _a.type) === common_1.VersioningType.URI) {
+        const versionOrVersions = this.getVersion(metadata);
+        if (versionOrVersions &&
+            ((_a = metadata.versioningOptions) === null || _a === void 0 ? void 0 : _a.type) === common_1.VersioningType.URI) {
             const versionPrefix = this.getVersionPrefix(metadata.versioningOptions);
-            // Version Neutral - Do not include version in URL
-            if (version !== common_1.VERSION_NEUTRAL) {
-                if (Array.isArray(version)) {
-                    paths = common_1.flatten(paths.map(path => version.map(v => path + `/${versionPrefix}${v}`)));
-                }
-                else {
-                    paths = paths.map(path => path + `/${versionPrefix}${version}`);
+            if (Array.isArray(versionOrVersions)) {
+                paths = common_1.flatten(paths.map(path => versionOrVersions.map(version => 
+                // Version Neutral - Do not include version in URL
+                version === common_1.VERSION_NEUTRAL
+                    ? path
+                    : `${path}/${versionPrefix}${version}`)));
+            }
+            else {
+                // Version Neutral - Do not include version in URL
+                if (versionOrVersions !== common_1.VERSION_NEUTRAL) {
+                    paths = paths.map(path => `${path}/${versionPrefix}${versionOrVersions}`);
                 }
             }
         }

package/router/router-execution-context.js

@@ -158,15 +158,16 @@ class RouterExecutionContext {
                 return await this.responseController.render(result, res, renderTemplate);
             };
         }
-        if (redirectResponse && typeof redirectResponse.url === 'string') {
+        if (redirectResponse && shared_utils_1.isString(redirectResponse.url)) {
             return async (result, res) => {
                 await this.responseController.redirect(result, res, redirectResponse);
             };
         }
         const isSseHandler = !!this.reflectSse(callback);
         if (isSseHandler) {
-            return async (result, res, req) => {
-                await this.responseController.sse(result, res.raw || res, req.raw || req);
+            return (result, res, req) => {
+                var _a;
+                this.responseController.sse(result, res.raw || res, req.raw || req, { additionalHeaders: (_a = res.getHeaders) === null || _a === void 0 ? void 0 : _a.call(res) });
             };
         }
         return async (result, res) => {

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

@@ -2,7 +2,7 @@
 import { HttpServer, RequestMethod } from '@nestjs/common';
 import { IncomingMessage } from 'http';
 import { Observable } from 'rxjs';
-import { HeaderStream } from './sse-stream';
+import { AdditionalHeaders, WritableHeaderStream } from './sse-stream';
 export interface CustomHeader {
     name: string;
     value: string;
@@ -13,6 +13,7 @@ export interface RedirectResponse {
 }
 export declare class RouterResponseController {
     private readonly applicationRef;
+    private readonly logger;
     constructor(applicationRef: HttpServer);
     apply<TInput = any, TResponse = any>(result: TInput, response: TResponse, httpStatusCode?: number): Promise<any>;
     redirect<TInput = any, TResponse = any>(resultOrDeferred: TInput, response: TResponse, redirectResponse: RedirectResponse): Promise<void>;
@@ -21,6 +22,8 @@ export declare class RouterResponseController {
     getStatusByMethod(requestMethod: RequestMethod): number;
     setHeaders<TResponse = unknown>(response: TResponse, headers: CustomHeader[]): void;
     setStatus<TResponse = unknown>(response: TResponse, statusCode: number): void;
-    sse<TInput extends Observable<unknown> = any, TResponse extends HeaderStream = any, TRequest extends IncomingMessage = any>(result: TInput, response: TResponse, request: TRequest): Promise<void>;
+    sse<TInput extends Observable<unknown> = any, TResponse extends WritableHeaderStream = any, TRequest extends IncomingMessage = any>(result: TInput, response: TResponse, request: TRequest, options?: {
+        additionalHeaders: AdditionalHeaders;
+    }): void;
     private assertObservable;
 }

package/router/router-response-controller.js

@@ -9,6 +9,7 @@ const sse_stream_1 = require("./sse-stream");
 class RouterResponseController {
     constructor(applicationRef) {
         this.applicationRef = applicationRef;
+        this.logger = new common_1.Logger(RouterResponseController.name);
     }
     async apply(result, response, httpStatusCode) {
         return this.applicationRef.reply(response, result, httpStatusCode);
@@ -47,21 +48,29 @@ class RouterResponseController {
     setStatus(response, statusCode) {
         this.applicationRef.status(response, statusCode);
     }
-    async sse(result, response, request) {
+    sse(result, response, request, options) {
         // It's possible that we sent headers already so don't use a stream
         if (response.writableEnded) {
             return;
         }
         this.assertObservable(result);
         const stream = new sse_stream_1.SseStream(request);
-        stream.pipe(response);
+        stream.pipe(response, options);
         const subscription = result
-            .pipe(operators_1.debounce((message) => new Promise(resolve => {
-            if (!shared_utils_1.isObject(message)) {
-                message = { data: message };
+            .pipe(operators_1.map((message) => {
+            if (shared_utils_1.isObject(message)) {
+                return message;
             }
-            stream.writeMessage(message, resolve);
-        })))
+            return { data: message };
+        }), operators_1.debounce(message => new Promise(resolve => stream.writeMessage(message, () => resolve()))), operators_1.catchError(err => {
+            const data = err instanceof Error ? err.message : err;
+            stream.writeMessage({ type: 'error', data }, writeError => {
+                if (writeError) {
+                    this.logger.error(writeError);
+                }
+            });
+            return rxjs_1.EMPTY;
+        }))
             .subscribe({
             complete: () => {
                 response.end();

package/router/sse-stream.d.ts

@@ -2,13 +2,18 @@
 import { MessageEvent } from '@nestjs/common/interfaces';
 import { IncomingMessage, OutgoingHttpHeaders } from 'http';
 import { Transform } from 'stream';
+export declare type AdditionalHeaders = Record<string, string[] | string | number | undefined>;
+interface ReadHeaders {
+    getHeaders?(): AdditionalHeaders;
+}
 interface WriteHeaders {
     writableEnded?: boolean;
     writeHead?(statusCode: number, reasonPhrase?: string, headers?: OutgoingHttpHeaders): void;
     writeHead?(statusCode: number, headers?: OutgoingHttpHeaders): void;
     flushHeaders?(): void;
 }
-export declare type HeaderStream = NodeJS.WritableStream & WriteHeaders;
+export declare type WritableHeaderStream = NodeJS.WritableStream & WriteHeaders;
+export declare type HeaderStream = WritableHeaderStream & ReadHeaders;
 /**
  * Adapted from https://raw.githubusercontent.com/EventSource/node-ssestream
  * Transforms "messages" to W3C event stream content.
@@ -25,7 +30,8 @@ export declare type HeaderStream = NodeJS.WritableStream & WriteHeaders;
 export declare class SseStream extends Transform {
     private lastEventId;
     constructor(req?: IncomingMessage);
-    pipe<T extends HeaderStream>(destination: T, options?: {
+    pipe<T extends WritableHeaderStream>(destination: T, options?: {
+        additionalHeaders?: AdditionalHeaders;
         end?: boolean;
     }): T;
     _transform(message: MessageEvent, encoding: string, callback: (error?: Error | null, data?: any) => void): void;

package/router/sse-stream.js

@@ -1,9 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SseStream = void 0;
+const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
 const stream_1 = require("stream");
 function toDataString(data) {
-    if (typeof data === 'object') {
+    if (shared_utils_1.isObject(data)) {
         return toDataString(JSON.stringify(data));
     }
     return data
@@ -36,17 +37,13 @@ class SseStream extends stream_1.Transform {
     }
     pipe(destination, options) {
         if (destination.writeHead) {
-            destination.writeHead(200, {
+            destination.writeHead(200, Object.assign(Object.assign({}, options === null || options === void 0 ? void 0 : options.additionalHeaders), { 
                 // See https://github.com/dunglas/mercure/blob/master/hub/subscribe.go#L124-L130
-                'Content-Type': 'text/event-stream',
-                Connection: 'keep-alive',
+                'Content-Type': 'text/event-stream', Connection: 'keep-alive', 
                 // Disable cache, even for old browsers and proxies
-                'Cache-Control': 'private, no-cache, no-store, must-revalidate, max-age=0, no-transform',
-                Pragma: 'no-cache',
-                Expire: '0',
+                'Cache-Control': 'private, no-cache, no-store, must-revalidate, max-age=0, no-transform', Pragma: 'no-cache', Expire: '0', 
                 // NGINX support https://www.nginx.com/resources/wiki/start/topics/examples/x-accel/#x-accel-buffering
-                'X-Accel-Buffering': 'no',
-            });
+                'X-Accel-Buffering': 'no' }));
             destination.flushHeaders();
         }
         destination.write(':\n');

package/router/utils/flatten-route-paths.util.js

@@ -11,7 +11,7 @@ function flattenRoutePaths(routes) {
         if (item.children) {
             const childrenRef = item.children;
             childrenRef.forEach(child => {
-                if (typeof child !== 'string' && child.path) {
+                if (!shared_utils_1.isString(child) && child.path) {
                     child.path = shared_utils_1.normalizePath(shared_utils_1.normalizePath(item.path) + shared_utils_1.normalizePath(child.path));
                 }
                 else {

package/scanner.d.ts

@@ -11,11 +11,12 @@ export declare class DependenciesScanner {
     private readonly container;
     private readonly metadataScanner;
     private readonly applicationConfig;
+    private readonly logger;
     private readonly applicationProvidersApplyMap;
     constructor(container: NestContainer, metadataScanner: MetadataScanner, applicationConfig?: ApplicationConfig);
     scan(module: Type<any>): Promise<void>;
     scanForModules(moduleDefinition: ForwardReference | Type<unknown> | DynamicModule | Promise<DynamicModule>, scope?: Type<unknown>[], ctxRegistry?: (ForwardReference | DynamicModule | Type<unknown>)[]): Promise<Module[]>;
-    insertModule(module: any, scope: Type<unknown>[]): Promise<Module | undefined>;
+    insertModule(moduleDefinition: any, scope: Type<unknown>[]): Promise<Module | undefined>;
     scanModulesForDependencies(modules?: Map<string, Module>): Promise<void>;
     reflectImports(module: Type<unknown>, token: string, context: string): Promise<void>;
     reflectProviders(module: Type<any>, token: string): void;
@@ -47,7 +48,22 @@ export declare class DependenciesScanner {
         [type: string]: Function;
     };
     isDynamicModule(module: Type<any> | DynamicModule): module is DynamicModule;
-    isForwardReference(module: Type<any> | DynamicModule | ForwardReference): module is ForwardReference;
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Injectable()` decorator.
+     */
+    private isInjectable;
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Controller()` decorator.
+     */
+    private isController;
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Catch()` decorator.
+     */
+    private isExceptionFilter;
+    private isForwardReference;
     private flatten;
     private isRequestOrTransient;
 }

package/scanner.js

@@ -11,6 +11,7 @@ const application_config_1 = require("./application-config");
 const constants_2 = require("./constants");
 const circular_dependency_exception_1 = require("./errors/exceptions/circular-dependency.exception");
 const invalid_module_exception_1 = require("./errors/exceptions/invalid-module.exception");
+const invalid_class_module_exception_1 = require("./errors/exceptions/invalid-class-module.exception");
 const undefined_module_exception_1 = require("./errors/exceptions/undefined-module.exception");
 const get_class_scope_1 = require("./helpers/get-class-scope");
 const internal_core_module_factory_1 = require("./injector/internal-core-module-factory");
@@ -19,6 +20,7 @@ class DependenciesScanner {
         this.container = container;
         this.metadataScanner = metadataScanner;
         this.applicationConfig = applicationConfig;
+        this.logger = new common_1.Logger(DependenciesScanner.name);
         this.applicationProvidersApplyMap = [];
     }
     async scan(module) {
@@ -65,11 +67,17 @@ class DependenciesScanner {
         }
         return [moduleInstance].concat(registeredModuleRefs);
     }
-    async insertModule(module, scope) {
-        if (module && module.forwardRef) {
-            return this.container.addModule(module.forwardRef(), scope);
+    async insertModule(moduleDefinition, scope) {
+        const moduleToAdd = this.isForwardReference(moduleDefinition)
+            ? moduleDefinition.forwardRef()
+            : moduleDefinition;
+        if (this.isInjectable(moduleToAdd) ||
+            this.isController(moduleToAdd) ||
+            this.isExceptionFilter(moduleToAdd)) {
+            // TODO(v9): Throw the exception instead of printing a warning
+            this.logger.warn(new invalid_class_module_exception_1.InvalidClassModuleException(moduleDefinition, scope).message);
         }
-        return this.container.addModule(module, scope);
+        return this.container.addModule(moduleToAdd, scope);
     }
     async scanModulesForDependencies(modules = this.container.getModules()) {
         for (const [token, { metatype }] of modules) {
@@ -179,7 +187,7 @@ class DependenciesScanner {
         if (shared_utils_1.isUndefined(related)) {
             throw new circular_dependency_exception_1.CircularDependencyException(context);
         }
-        if (related && related.forwardRef) {
+        if (this.isForwardReference(related)) {
             return this.container.addImport(related.forwardRef(), token);
         }
         await this.container.addImport(related, token);
@@ -288,6 +296,27 @@ class DependenciesScanner {
     isDynamicModule(module) {
         return module && !!module.module;
     }
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Injectable()` decorator.
+     */
+    isInjectable(metatype) {
+        return !!Reflect.getMetadata(constants_1.INJECTABLE_WATERMARK, metatype);
+    }
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Controller()` decorator.
+     */
+    isController(metatype) {
+        return !!Reflect.getMetadata(constants_1.CONTROLLER_WATERMARK, metatype);
+    }
+    /**
+     * @param metatype
+     * @returns `true` if `metatype` is annotated with the `@Catch()` decorator.
+     */
+    isExceptionFilter(metatype) {
+        return !!Reflect.getMetadata(constants_1.CATCH_WATERMARK, metatype);
+    }
     isForwardReference(module) {
         return module && !!module.forwardRef;
     }