Merge pull request #11186 from CodyTseng/fix-ws-adaptor-filter-non-ni…

…l-result fix(ws): filter only nil results
nestjs · Mar 10, 2023 · 797b088 · 797b088
2 parents 8b72c52 + 6f8f111
commit 797b088
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/platform-ws/adapters/ws-adapter.ts b/packages/platform-ws/adapters/ws-adapter.ts
@@ -1,5 +1,6 @@
 import { INestApplicationContext, Logger } from '@nestjs/common';
 import { loadPackage } from '@nestjs/common/utils/load-package.util';
+import { isNil } from '@nestjs/common/utils/shared.utils';
 import { AbstractWsAdapter } from '@nestjs/websockets';
 import {
   CLOSE_EVENT,
@@ -109,7 +110,7 @@ export class WsAdapter extends AbstractWsAdapter {
     const source$ = fromEvent(client, 'message').pipe(
       mergeMap(data =>
         this.bindMessageHandler(data, handlers, transform).pipe(
-          filter(result => result),
+          filter(result => !isNil(result)),
         ),
       ),
       takeUntil(close$),