summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorZane Helton <zane99@me.com>2025-06-29 18:42:15 -0400
committerRory& <root@rory.gay>2025-09-29 00:16:23 +0200
commit06ae335d1b52b21b6da064a1bbcb5d7aad253b61 (patch)
treedcb99da1dddee24d4ecccfe234a27ad02e51e38a /src
parentfix error (diff)
downloadserver-ts-06ae335d1b52b21b6da064a1bbcb5d7aad253b61.tar.xz
Remove @everyone role from `onIdentify`
This prevents the @everyone role from leaking into places that might
display it on the frontend. The @everyone role is still visible when
needed, but `merged_members` (aka GuildMembers) won't include the role.

Resolves: #1213
Diffstat (limited to 'src')
-rw-r--r--src/gateway/opcodes/Identify.ts5
1 files changed, 4 insertions, 1 deletions
diff --git a/src/gateway/opcodes/Identify.ts b/src/gateway/opcodes/Identify.ts

index 5e2e28a24..9543a3975 100644 --- a/src/gateway/opcodes/Identify.ts +++ b/src/gateway/opcodes/Identify.ts
@@ -244,7 +244,10 @@ export async function onIdentify(this: WebSocket, data: Payload) { return [ { ...x, - roles: x.roles.map((x) => x.id), + // filter out @everyone role + roles: x.roles + .filter((r) => r.id !== x.guild.id) + .map((x) => x.id), // add back user, which we don't fetch from db // TODO: For guild profiles, this may need to be changed.