mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Prevent sending shadowing healthcheck on a closed connection

Browse source 
Co-authored-by: Sergey Skrobotov <sergey@signal.org>
This commit is contained in:
automated-signal 2024-04-12 18:51:28 -05:00 • committed by GitHub
parent 95f2567ccc
commit d586a75819
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

9
ts/textsecure/WebsocketResources.ts
View file

@ -447,6 +447,7 @@ export class WebSocketResourceWithShadowing implements IWebSocketResource {
this.main.close();
if (this.shadowing) {
this.shadowing.close();
this.shadowing = undefined;
} else {
this.shadowingConnection.abort();
}
@ -456,6 +457,7 @@ export class WebSocketResourceWithShadowing implements IWebSocketResource {
this.main.shutdown();
if (this.shadowing) {
this.shadowing.shutdown();
this.shadowing = undefined;
} else {
this.shadowingConnection.abort();
}
@ -482,11 +484,12 @@ export class WebSocketResourceWithShadowing implements IWebSocketResource {
}
private async sendShadowRequest(): Promise<void> {
// it could be that we're still connecting libsignal websocket
// in which case we're skipping the check
// In the shadowing mode, it could be that we're either
// still connecting libsignal websocket or have already closed it.
// In those cases we're not running shadowing check.
if (!this.shadowing) {
log.info(
`${this.logId}: skipping healthcheck - websocket not connected yet`
`${this.logId}: skipping healthcheck - websocket not connected or already closed`
);
return;
}