Page MenuHomePhabricator
Files F3397836

D6932.diff
No OneTemporary
Actions

D6932.diff
View Options

diff --git a/keyserver/src/fetchers/message-fetchers.js b/keyserver/src/fetchers/message-fetchers.js
--- a/keyserver/src/fetchers/message-fetchers.js
+++ b/keyserver/src/fetchers/message-fetchers.js
@@ -708,6 +708,16 @@
return rawMessageInfoFromRows(result, viewer, derivedMessages);
}
+async function fetchThreadMessagesCount(threadID: string): Promise<number> {
+ const query = SQL`
+ SELECT COUNT(*) AS count
+ FROM messages
+ WHERE thread = ${threadID}
+ `;
+ const [result] = await dbQuery(query);
+ return result[0].count;
+}
+
export {
fetchCollapsableNotifs,
fetchMessageInfos,
@@ -716,4 +726,5 @@
fetchMessageInfoForLocalID,
fetchMessageInfoForEntryAction,
fetchMessageInfoByID,
+ fetchThreadMessagesCount,
};
diff --git a/keyserver/src/responders/message-responders.js b/keyserver/src/responders/message-responders.js
--- a/keyserver/src/responders/message-responders.js
+++ b/keyserver/src/responders/message-responders.js
@@ -36,6 +36,7 @@
fetchMessageInfos,
fetchMessageInfoForLocalID,
fetchMessageInfoByID,
+ fetchThreadMessagesCount,
} from '../fetchers/message-fetchers.js';
import { fetchServerThreadInfos } from '../fetchers/thread-fetchers.js';
import { checkThreadPermission } from '../fetchers/thread-permission-fetchers.js';
@@ -90,7 +91,16 @@
messageData = { ...messageData, localID };
}
if (sidebarCreation) {
- messageData = { ...messageData, sidebarCreation };
+ const numMessages = await fetchThreadMessagesCount(threadID);
+ if (numMessages === 2) {
+ // sidebarCreation is set below to prevent double notifs from a sidebar
+ // creation. We expect precisely two messages to appear before a
+ // sidebarCreation: a SIDEBAR_SOURCE and a CREATE_SIDEBAR. If two users
+ // attempt to create a sidebar at the same time, then both clients will
+ // attempt to set sidebarCreation here, but we only want to suppress
+ // notifs for the client that won the race.
+ messageData = { ...messageData, sidebarCreation };
+ }
}
const rawMessageInfos = await createMessages(viewer, [messageData]);

File Metadata

Mime Type
text/plain
Expires
Mon, Dec 2, 7:40 PM (20 h, 55 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2608714
Default Alt Text
D6932.diff (2 KB)

Event Timeline