diff options
author | AlTech98 <altech123159@gmail.com> | 2021-09-02 19:33:04 +0200 |
---|---|---|
committer | AlTech98 <altech123159@gmail.com> | 2021-09-02 19:33:04 +0200 |
commit | 8ff56b1f9e1b02c111ae725916f7e5be63d94662 (patch) | |
tree | 56907eb2747a362d26db4cfd6494bcaee234e7e6 /gateway | |
parent | :art: update openapi title (diff) | |
download | server-8ff56b1f9e1b02c111ae725916f7e5be63d94662.tar.xz |
VOICE_STATE_UPDATE implementation, fix #210
Diffstat (limited to 'gateway')
-rw-r--r-- | gateway/src/events/Close.ts | 6 | ||||
-rw-r--r-- | gateway/src/events/Connection.ts | 3 | ||||
-rw-r--r-- | gateway/src/opcodes/Identify.ts | 26 | ||||
-rw-r--r-- | gateway/src/opcodes/VoiceStateUpdate.ts | 67 | ||||
-rw-r--r-- | gateway/src/schema/VoiceStateUpdateSchema.ts (renamed from gateway/src/schema/VoiceStateUpdate.ts.ts) | 4 | ||||
-rw-r--r-- | gateway/src/util/SessionUtils.ts | 11 | ||||
-rw-r--r-- | gateway/src/util/WebSocket.ts | 1 |
7 files changed, 95 insertions, 23 deletions
diff --git a/gateway/src/events/Close.ts b/gateway/src/events/Close.ts index f1a8fa9b..26f4e7df 100644 --- a/gateway/src/events/Close.ts +++ b/gateway/src/events/Close.ts @@ -1,7 +1,9 @@ -import WebSocket from "ws"; +import WebSocket from "../util/WebSocket"; import { Message } from "./Message"; +import {Session} from "@fosscord/util"; -export function Close(this: WebSocket, code: number, reason: string) { +export async function Close(this: WebSocket, code: number, reason: string) { + await Session.delete({session_id: this.session_id}) // @ts-ignore this.off("message", Message); } diff --git a/gateway/src/events/Connection.ts b/gateway/src/events/Connection.ts index fed3c611..1af484eb 100644 --- a/gateway/src/events/Connection.ts +++ b/gateway/src/events/Connection.ts @@ -7,6 +7,7 @@ import { Send } from "../util/Send"; import { CLOSECODES, OPCODES } from "../util/Constants"; import { createDeflate } from "zlib"; import { URL } from "url"; +import {Session} from "@fosscord/util"; var erlpack: any; try { erlpack = require("erlpack"); @@ -56,10 +57,12 @@ export async function Connection(this: Server, socket: WebSocket, request: Incom }); socket.readyTimeout = setTimeout(() => { + Session.delete({session_id: socket.session_id}) //should we await? return socket.close(CLOSECODES.Session_timed_out); }, 1000 * 30); } catch (error) { console.error(error); + Session.delete({session_id: socket.session_id}) //should we await? return socket.close(CLOSECODES.Unknown_error); } } diff --git a/gateway/src/opcodes/Identify.ts b/gateway/src/opcodes/Identify.ts index 958f1b73..3984be73 100644 --- a/gateway/src/opcodes/Identify.ts +++ b/gateway/src/opcodes/Identify.ts @@ -8,6 +8,7 @@ import { Member, ReadyEventData, User, + Session, EVENTEnum, Config, dbConnection, @@ -18,8 +19,8 @@ import { Send } from "../util/Send"; // import experiments from "./experiments.json"; const experiments: any = []; import { check } from "./instanceOf"; -import { Like } from "../../../util/node_modules/typeorm"; import { Recipient } from "../../../util/dist/entities/Recipient"; +import { genSessionId } from "../util/SessionUtils"; // TODO: bot sharding // TODO: check priviliged intents @@ -73,6 +74,22 @@ export async function onIdentify(this: WebSocket, data: Payload) { const user = await User.findOneOrFail({ id: this.user_id }); if (!user) return this.close(CLOSECODES.Authentication_failed); + const session_id = genSessionId(); + this.session_id = session_id; //Set the session of the WebSocket object + const session = new Session({ + user_id: this.user_id, + session_id: session_id, + status: "online", //does the session always start as online? + client_info: { //TODO read from identity + client: "desktop", + os: "linux", + version: 0 + } + }); + + //We save the session and we delete it when the websocket is closed + await session.save(); + const public_user = { username: user.username, discriminator: user.discriminator, @@ -135,7 +152,7 @@ export async function onIdentify(this: WebSocket, data: Payload) { version: 642, }, private_channels: channels, - session_id: "", // TODO + session_id: session_id, analytics_token: "", // TODO connected_accounts: [], // TODO consents: { @@ -164,5 +181,10 @@ export async function onIdentify(this: WebSocket, data: Payload) { d, }); + //TODO send READY_SUPPLEMENTAL + //TODO send GUILD_MEMBER_LIST_UPDATE + //TODO send SESSIONS_REPLACE + //TODO send VOICE_STATE_UPDATE to let the client know if another device is already connected to a voice channel + await setupListener.call(this); } diff --git a/gateway/src/opcodes/VoiceStateUpdate.ts b/gateway/src/opcodes/VoiceStateUpdate.ts index 0d51513d..4719949c 100644 --- a/gateway/src/opcodes/VoiceStateUpdate.ts +++ b/gateway/src/opcodes/VoiceStateUpdate.ts @@ -1,26 +1,59 @@ -import { VoiceStateUpdateSchema } from "../schema/VoiceStateUpdate.ts"; -import { CLOSECODES, Payload } from "../util/Constants"; -import { Send } from "../util/Send"; - +import { VoiceStateUpdateSchema } from "../schema/VoiceStateUpdateSchema"; +import { Payload } from "../util/Constants"; import WebSocket from "../util/WebSocket"; import { check } from "./instanceOf"; -// TODO: implementation +import { Config, emitEvent, VoiceServerUpdateEvent, VoiceState, VoiceStateUpdateEvent } from "@fosscord/util"; +import { genVoiceToken } from "../util/SessionUtils"; // TODO: check if a voice server is setup -// TODO: save voice servers in database and retrieve them // Notice: Bot users respect the voice channel's user limit, if set. When the voice channel is full, you will not receive the Voice State Update or Voice Server Update events in response to your own Voice State Update. Having MANAGE_CHANNELS permission bypasses this limit and allows you to join regardless of the channel being full or not. export async function onVoiceStateUpdate(this: WebSocket, data: Payload) { check.call(this, VoiceStateUpdateSchema, data.d); const body = data.d as VoiceStateUpdateSchema; - await Send(this, { - op: 0, - s: this.sequence++, - t: "VOICE_SERVER_UPDATE", - d: { - token: ``, - guild_id: body.guild_id, - endpoint: `localhost:3004`, - }, - }); -} + let voiceState + try { + voiceState = await VoiceState.findOneOrFail({where:{ user_id: this.user_id },relations: ["member", "member.user", "member.roles"]}); + if(voiceState.session_id !== this.session_id && body.channel_id === null) { //Should we also check guild_id === null? + //changing deaf or mute on a client that's not the one with the same session of the voicestate in the database should be ignored + return + } + + //The event send by Discord's client on channel leave has both guild_id and channel_id as null + if(body.guild_id === null) body.guild_id = voiceState.guild_id; + voiceState.assign(body); + } catch (error) { + voiceState = new VoiceState({ + ...body, + user_id: this.user_id, + deaf: false, + mute: false, + suppress: false + }) + } + + //If the session changed we generate a new token + if(voiceState.session_id !== this.session_id) + voiceState.token = genVoiceToken() + voiceState.session_id = this.session_id + + //TODO the member should only have these properties: hoisted_role, deaf, joined_at, mute, roles, user + //TODO the member.user should only have these properties: avatar, discriminator, id, username + const {id, ...newObj} = voiceState; + + await Promise.all([ + voiceState.save(), + emitEvent({ event: "VOICE_STATE_UPDATE", data: newObj, guild_id: voiceState.guild_id} as VoiceStateUpdateEvent), + ]); + + //If it's null it means that we are leaving the channel and this event is not needed + if(voiceState.channel_id !== null) { + const regions = Config.get().regions; + + await emitEvent({ event: "VOICE_SERVER_UPDATE", data: { + token: voiceState.token, + guild_id: voiceState.guild_id, + endpoint: regions.available[0].endpoint, //TODO return best endpoint or default + }, guild_id: voiceState.guild_id } as VoiceServerUpdateEvent) + } +} \ No newline at end of file diff --git a/gateway/src/schema/VoiceStateUpdate.ts.ts b/gateway/src/schema/VoiceStateUpdateSchema.ts index 4345c2f6..9efa191e 100644 --- a/gateway/src/schema/VoiceStateUpdate.ts.ts +++ b/gateway/src/schema/VoiceStateUpdateSchema.ts @@ -1,6 +1,6 @@ export const VoiceStateUpdateSchema = { $guild_id: String, - channel_id: String, + $channel_id: String, self_mute: Boolean, self_deaf: Boolean, self_video: Boolean, @@ -8,7 +8,7 @@ export const VoiceStateUpdateSchema = { export interface VoiceStateUpdateSchema { guild_id?: string; - channel_id: string; + channel_id?: string; self_mute: boolean; self_deaf: boolean; self_video: boolean; diff --git a/gateway/src/util/SessionUtils.ts b/gateway/src/util/SessionUtils.ts new file mode 100644 index 00000000..1ca23316 --- /dev/null +++ b/gateway/src/util/SessionUtils.ts @@ -0,0 +1,11 @@ +export function genSessionId() { + return genRanHex(32) +} + +export function genVoiceToken() { + return genRanHex(16) +} + +function genRanHex(size: number) { + return [...Array(size)].map(() => Math.floor(Math.random() * 16).toString(16)).join(''); +} \ No newline at end of file diff --git a/gateway/src/util/WebSocket.ts b/gateway/src/util/WebSocket.ts index d1e13555..2c763743 100644 --- a/gateway/src/util/WebSocket.ts +++ b/gateway/src/util/WebSocket.ts @@ -6,6 +6,7 @@ import { Channel } from "amqplib"; interface WebSocket extends WS { version: number; user_id: string; + session_id: string; encoding: "etf" | "json"; compress?: "zlib-stream"; shard_count?: bigint; |