andras/vault-link
andras/vault-link
1
0
Fork 0
Code Issues Pull requests 19 Projects Releases 55 Packages Wiki Activity Actions 24

Various improvements #169

Merged
schmelczer merged 78 commits from asch/saturday into main 2025-11-30 15:24:52 +00:00
Conversation 2 Commits 78 Files changed 112 +7 -2
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit bd5a620942 - Show all commits

Don't broadcast without clients

Prev Next
Andras Schmelczer 2025-11-29 17:26:09 +00:00
commit bd5a620942

9
sync-server/src/app_state/websocket/broadcasts.rs
View file

@ -1,7 +1,7 @@
use std::{collections::HashMap, sync::Arc}; use std::{collections::HashMap, sync::Arc};
use anyhow::Context; use anyhow::Context;
use log::warn; use log::{debug, warn};
use tokio::sync::{Mutex, broadcast}; use tokio::sync::{Mutex, broadcast};
use super::models::WebSocketServerMessageWithOrigin; use super::models::WebSocketServerMessageWithOrigin;
@ -39,7 +39,12 @@ impl Broadcasts {
vault: VaultId, vault: VaultId,
document: WebSocketServerMessageWithOrigin, document: WebSocketServerMessageWithOrigin,
) { ) {
let tx = self.get_or_create(vault).await; let tx = self.get_or_create(vault.clone()).await;
if tx.receiver_count() == 0 {
debug!("Skipping broadcast, no clients connected for vault `{vault}`");
return;
}
let result = tx let result = tx
.send(document) .send(document)