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 {