Skip to content

Commit

Permalink
fix: forward the local flag to the adapter when using fetchSockets()
Browse files Browse the repository at this point in the history
  • Loading branch information
darrachequesne committed May 3, 2022
1 parent 9b43c91 commit 30430f0
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions lib/broadcast-operator.ts
Expand Up @@ -262,6 +262,7 @@ export class BroadcastOperator<EmitEvents extends EventsMap, SocketData>
.fetchSockets({
rooms: this.rooms,
except: this.exceptRooms,
flags: this.flags,
})
.then((sockets) => {
return sockets.map((socket) => {
Expand Down Expand Up @@ -289,6 +290,7 @@ export class BroadcastOperator<EmitEvents extends EventsMap, SocketData>
{
rooms: this.rooms,
except: this.exceptRooms,
flags: this.flags,
},
Array.isArray(room) ? room : [room]
);
Expand All @@ -305,6 +307,7 @@ export class BroadcastOperator<EmitEvents extends EventsMap, SocketData>
{
rooms: this.rooms,
except: this.exceptRooms,
flags: this.flags,
},
Array.isArray(room) ? room : [room]
);
Expand All @@ -321,6 +324,7 @@ export class BroadcastOperator<EmitEvents extends EventsMap, SocketData>
{
rooms: this.rooms,
except: this.exceptRooms,
flags: this.flags,
},
close
);
Expand Down

4 comments on commit 30430f0

@seankarson
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@darrachequesne Any chance you could trigger a release that includes this change? I'd really appreciate it, and I'm happy to help in any way I can!

@schw4rzlicht
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd love to see this one released as well :)

@derikb
Copy link

@derikb derikb commented on 30430f0 May 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

me as well

@darrachequesne
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.