Skip to content

Commit ccf52de

Browse files
committed
fix: improve async options provider by using nullish coalescing and ensuring proper async handling
1 parent ba33c21 commit ccf52de

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

packages/node-redis/lib/redis.providers.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export const createAsyncOptionsProvider = (options: RedisModuleAsyncOptions): Pr
3737
return {
3838
provide: MODULE_OPTIONS_TOKEN,
3939
useFactory: options.useFactory,
40-
inject: options.inject || []
40+
inject: options.inject ?? []
4141
};
4242
}
4343

@@ -50,8 +50,9 @@ export const createAsyncOptionsProvider = (options: RedisModuleAsyncOptions): Pr
5050

5151
return {
5252
provide: MODULE_OPTIONS_TOKEN,
53-
useFactory: async (optionsFactory: RedisOptionsFactory<RedisModuleOptions>) =>
54-
optionsFactory.createRedisOptions(),
55-
inject,
53+
useFactory: async (optionsFactory: RedisOptionsFactory<RedisModuleOptions>) => {
54+
return await optionsFactory.createRedisOptions();
55+
},
56+
inject
5657
};
5758
};

0 commit comments

Comments
 (0)