Merge pull request #13083 from nestjs/fix/non-unique-hashes

fix: generate unique hashes for dynamic modules, and clients
nestjs · Jan 23, 2024 · 829f57f · 829f57f
2 parents bfca362 + 9e8f25d
commit 829f57f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/common/module-utils/configurable-module.builder.ts b/packages/common/module-utils/configurable-module.builder.ts
@@ -209,7 +209,7 @@ export class ConfigurableModuleBuilder<
         const providers: Array<Provider> = [
           {
             provide: self.options.optionsInjectionToken,
-            useFactory: () => this.omitExtras(options, self.extras),
+            useValue: this.omitExtras(options, self.extras),
           },
         ];
         if (self.options.alwaysTransient) {

diff --git a/packages/microservices/module/clients.module.ts b/packages/microservices/module/clients.module.ts
@@ -20,8 +20,7 @@ export class ClientsModule {
     const clients = (clientsOptions || []).map(item => {
       return {
         provide: item.name,
-        useFactory: () =>
-          this.assignOnAppShutdownHook(ClientProxyFactory.create(item)),
+        useValue: this.assignOnAppShutdownHook(ClientProxyFactory.create(item)),
       };
     });
     return {