Implement Media Sink Wants in Voice #898
Conversation
|
i'm curious, are there any resources to read up on this feature? it isn't documented in the discord developer portal yet |
It isn't documented, just like voice receive (which is supported by kord voice), this was discovered through reverse engineering the official Discord Client. |
| @@ -21,6 +21,7 @@ public class DefaultVoiceGatewayBuilder( | |||
| public var client: HttpClient? = null | |||
| public var reconnectRetry: Retry? = null | |||
| public var eventFlow: MutableSharedFlow<VoiceEvent> = MutableSharedFlow(extraBufferCapacity = Int.MAX_VALUE) | |||
| public var isDeaf: Boolean = false | |||
There was a problem hiding this comment.
VoiceConnectionBuilder already has the selfDeaf option, can't this be used instead?
There was a problem hiding this comment.
I suppose. I think the VoiceConnectionBuilder could get cleaned up quite a bit though.
Like I state in the PR description, you can set selfDeaf and a streams implementation.
Since selfDeaf is just a UI update right now it's not that much of an issue but if it gets updated to actually prevent voice receive it might be a good idea to set it depending on the streams implementation so that there isn't any confusion.
And maybe add some more functionality to it as well:
interface Streams {
suspend fun setDeafen(value: Boolean)
suspend fun setMute(ssrc: UInt, value: Boolean)
}
There was a problem hiding this comment.
I think this is a fair solution. Makes the deafening much behavior much more consistent, especially with this addition. Streams#setDeafen might not be needed, as it isn't provided for the UI deafen either. I could see the use for a VoiceConnection#(set)Deafen which would UI deafen and media sink deafen (if streams are open).
|
Functionality-wise LGTM. |
Adds a new voice gateway command (op code
15) that controls whether the voice server sends voice packets to the bot.Still a bit unsure on the API since Kord voice allows setting the deafen value when joining and the streams implementation to use.