Skip to content

Commit c9621c1

Browse files
Merge pull request nestjs#237 from hicdre/master
Map key should not be object
2 parents 81a75fc + 15403ba commit c9621c1

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

src/websockets/container.ts

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,18 @@
11
import { WebSocketServerData, ObservableSocketServer } from './interfaces';
22

33
export class SocketsContainer {
4-
private readonly observableServers = new Map<WebSocketServerData, ObservableSocketServer>();
4+
private readonly observableServers = new Map<string, ObservableSocketServer>();
55

6-
public getAllServers(): Map<WebSocketServerData, ObservableSocketServer> {
6+
public getAllServers(): Map<string, ObservableSocketServer> {
77
return this.observableServers;
88
}
99

1010
public getServer(namespace: string, port: number): ObservableSocketServer {
11-
return this.observableServers.get({
12-
namespace,
13-
port,
14-
});
11+
return this.observableServers.get(`${namespace}:${port}`);
1512
}
1613

1714
public addServer(namespace: string, port: number, server: ObservableSocketServer) {
18-
this.observableServers.set({
19-
namespace,
20-
port,
21-
}, server);
15+
this.observableServers.set(`${namespace}:${port}`, server);
2216
}
2317

2418
public clear() {

0 commit comments

Comments
 (0)