bugfix(@nestjs/microservices) can't select/get from context

nestjs · Feb 16, 2018 · 03216d2 · 03216d2
1 parent 2b29461
commit 03216d2
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 4.6.4
+### Bug Fixes
+- **microservices**: can't select/get from context when using `NestFactory.createMicroservice` [#398](https://github.com/nestjs/nest/issues/398)
+
 ## 4.6.3
 ### Bug Fixes
 - **core**: error thrown when connecting to microservice (hybrid applicaton) [#425](https://github.com/nestjs/nest/issues/425)

diff --git a/lib/core/nest-application.js b/lib/core/nest-application.js
@@ -17,6 +17,7 @@ const iterare_1 = require("iterare");
 const logger_service_1 = require("@nestjs/common/services/logger.service");
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
 const express_adapter_1 = require("./adapters/express-adapter");
+const application_config_1 = require("./application-config");
 const constants_1 = require("./constants");
 const middlewares_module_1 = require("./middlewares/middlewares-module");
 const routes_resolver_1 = require("./router/routes-resolver");
@@ -114,7 +115,8 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
         if (!NestMicroservice) {
             throw new microservices_package_not_found_exception_1.MicroservicesPackageNotFoundException();
         }
-        const instance = new NestMicroservice(this.container, config);
+        const applicationConfig = new application_config_1.ApplicationConfig();
+        const instance = new NestMicroservice(this.container, config, applicationConfig);
         instance.setupListeners();
         instance.setIsInitialized(true);
         instance.setIsInitHookCalled(true);

diff --git a/src/core/nest-application-context.ts b/src/core/nest-application-context.ts
@@ -14,6 +14,11 @@ export class NestApplicationContext implements INestApplicationContext {
     protected contextModule,
   ) {}
 
+  public selectContextModule() {
+    const modules = this.container.getModules().values();
+    this.contextModule = modules.next().value;
+  }
+
   public select<T>(module: Metatype<T>): INestApplicationContext {
     const modules = this.container.getModules();
     const moduleMetatype = this.contextModule.metatype;

diff --git a/src/core/nest-application.ts b/src/core/nest-application.ts
@@ -82,11 +82,6 @@ export class NestApplication extends NestApplicationContext
     );
   }
 
-  public selectContextModule() {
-    const modules = this.container.getModules().values();
-    this.contextModule = modules.next().value;
-  }
-
   public applyOptions() {
     if (!this.appOptions) {
       return undefined;

diff --git a/src/microservices/nest-microservice.ts b/src/microservices/nest-microservice.ts
@@ -58,6 +58,8 @@ export class NestMicroservice extends NestApplicationContext
     this.server = strategy
       ? strategy
       : ServerFactory.create(this.microserviceConfig);
+
+    this.selectContextModule();
   }
 
   public setupModules() {