DiscordPHP icon indicating copy to clipboard operation
DiscordPHP copied to clipboard

Voice Client & Packets v2

Open alexandre433 opened this issue 8 months ago • 3 comments

In this pull request, i've updated the current methods & systems to use the newer version of Discord's Gateway version, to the most recent v8

The decryption was also updated to the most recent & preferred by discord aead_aes256_gcm_rtpsize

I've also updated a package that, in my eyes, was really old, and needed some extra love, in the case of, adding some php 8+ features & a Enum to help with some values & strings the trafficcophp/bytebuffer(forked)

On the Voice Packet side I've only been able to store the packets into a temporary file, since I'm mainly working with Windows, and not with WSL.

Any changes that will be required, hit me up!

alexandre433 avatar Mar 12 '25 16:03 alexandre433

I believe this would close #1290 as well

valzargaming avatar Apr 09 '25 13:04 valzargaming

I believe the only thing this is waiting on internally is some discussions revolving around the bytebuffer fork. Would it be possible to implement this PR either without changing the existing dependencies of our library, or adding them later in a separate PR?

valzargaming avatar May 16 '25 16:05 valzargaming

It is possible yes. I only made the change to update that package. Since it was a bit outdated, wanted to make Enums and such in it.

alexandre433 avatar May 31 '25 17:05 alexandre433

Any news on this? 💌

oliverschloebe avatar Jul 01 '25 10:07 oliverschloebe

To quote the author, "I've been busy lately with some changes at work. Whenever I have some more free time i can go back to the voice part. And finish it once and for all"

valzargaming avatar Jul 01 '25 10:07 valzargaming

v2.0 of the Voice system done. I'd say it's still in "BETA" phase since the recorded sound has a bit of static in it still. Haven't tested the output of sounds/playing sounds.

alexandre433 avatar Jul 06 '25 22:07 alexandre433

Marking this as a draft for now. I'm using this as a baseline when updating the existing implementation, but didn't want to set an expectation that this PR will be merged as-is.

valzargaming avatar Sep 30 '25 15:09 valzargaming