github/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2024-12-23 00:12:13 +00:00
Code Issues Projects Releases Packages Wiki Activity

fix: disconnect websocket if trying to send over closed connection

Browse source 
Signed-off-by: Tilman Vatteroth <git@tilmanvatteroth.de>
This commit is contained in:
Tilman Vatteroth 2023-03-26 15:16:18 +02:00
parent 57701b5d6c
commit ef699a9123
1 changed files with 16 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
commons/src/message-transporters/websocket-transporter.ts
View file

@ -28,7 +28,7 @@ export class WebsocketTransporter extends MessageTransporter {
this.websocket = websocket
this.bindWebsocketEvents(websocket)
if (this.websocket.readyState === WebSocket.OPEN) {
if (this.isConnected()) {
this.onConnected()
} else {
this.websocket.addEventListener('open', this.onConnected.bind(this))
@ -69,13 +69,26 @@ export class WebsocketTransporter extends MessageTransporter {
}
protected onDisconnecting() {
if (this.websocket === undefined) {
return
}
this.undbindEventsFromPreviousWebsocket()
this.websocket = undefined
super.onDisconnecting()
}
public sendMessage(content: Message<MessageType>): void {
if (this.websocket?.readyState !== WebSocket.OPEN) {
throw new Error("Can't send message over non-open socket")
if (!this.isConnected()) {
this.onDisconnecting()
console.debug(
"Can't send message over closed connection. Triggering onDisconencted event. Message that couldn't be sent was",
content
)
return
}
if (this.websocket === undefined) {
throw new Error('websocket transporter has no websocket connection')
}
try {