Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to voice gateway v8 #2717

Merged
merged 4 commits into from
Aug 21, 2024
Merged

Conversation

MinnDevelopment
Copy link
Member

@MinnDevelopment MinnDevelopment commented Aug 19, 2024

Pull Request Etiquette

Changes

  • Internal code
  • Library interface (affecting end-user code)
  • Documentation
  • Other: _____

Closes Issue: NaN

Description

Upgrades our voice gateway logic to v8.

Reference: discord/discord-api-docs#7075

TODO: Support new encryption modes

  • aead_aes256_gcm_rtpsize
  • aead_xchacha20_poly1305_rtpsize

@MinnDevelopment MinnDevelopment marked this pull request as draft August 19, 2024 22:53
@MinnDevelopment MinnDevelopment marked this pull request as ready for review August 20, 2024 21:01
@MinnDevelopment MinnDevelopment merged commit c5b0a6d into master Aug 21, 2024
1 check passed
@MinnDevelopment MinnDevelopment deleted the feature/voice-gateway-v8 branch August 21, 2024 11:20
mergify bot referenced this pull request in SvenKirschbaum/musikbot-client Aug 24, 2024
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|---|---|
| maven | stage | patch | `3.9.8-amazoncorretto-21` -> `3.9.9-amazoncorretto-21` | [![age](https://developer.mend.io/api/mc/badges/age/docker/maven/3.9.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/docker/maven/3.9.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/docker/maven/3.9.8/3.9.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/maven/3.9.8/3.9.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) |
| [net.dv8tion:JDA](https://togithub.com/discord-jda/JDA) | compile | minor | `5.0.2` -> `5.1.0` | [![age](https://developer.mend.io/api/mc/badges/age/maven/net.dv8tion:JDA/5.1.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/net.dv8tion:JDA/5.1.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/net.dv8tion:JDA/5.0.2/5.1.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/net.dv8tion:JDA/5.0.2/5.1.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |
| [org.springframework.boot:spring-boot-starter-parent](https://spring.io/projects/spring-boot) ([source](https://togithub.com/spring-projects/spring-boot)) | parent | patch | `3.3.2` -> `3.3.3` | [![age](https://developer.mend.io/api/mc/badges/age/maven/org.springframework.boot:spring-boot-starter-parent/3.3.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/org.springframework.boot:spring-boot-starter-parent/3.3.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/org.springframework.boot:spring-boot-starter-parent/3.3.2/3.3.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/org.springframework.boot:spring-boot-starter-parent/3.3.2/3.3.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>discord-jda/JDA (net.dv8tion:JDA)</summary>

### [`v5.1.0`](https://togithub.com/discord-jda/JDA/releases/tag/v5.1.0): | Voice Gateway v8

### Overview

This release updates the implementation of the voice gateway to API version 8 (previously 4). Previous versions will be incompatible coming November 18, 2024. This includes adding a new dependency on [tink](https://togithub.com/tink-crypto/tink-java), to support the new encryption modes.

Additionally, the `MessageEmbedEvent` has been removed. Discord sends standard update events for embed loading now, this event can no longer be supported.

#### New Features

-   Add new message and embed types by [@&#8203;MinnDevelopment](https://togithub.com/MinnDevelopment) in [https://github.com/discord-jda/JDA/pull/2716](https://togithub.com/discord-jda/JDA/pull/2716)

#### Changes

-   Upgrade to voice gateway v8 by [@&#8203;MinnDevelopment](https://togithub.com/MinnDevelopment) in [https://github.com/discord-jda/JDA/pull/2717](https://togithub.com/discord-jda/JDA/pull/2717)
-   Remove MessageEmbedEvent by [@&#8203;MinnDevelopment](https://togithub.com/MinnDevelopment) in [https://github.com/discord-jda/JDA/pull/2720](https://togithub.com/discord-jda/JDA/pull/2720)

#### Bug Fixes

-   Handle guild stickers array as optional in EntityBuilder#createGuild by [@&#8203;Xirado](https://togithub.com/Xirado) in [https://github.com/discord-jda/JDA/pull/2714](https://togithub.com/discord-jda/JDA/pull/2714)

**Full Changelog**: discord-jda/JDA@v5.0.2...v5.1.0

### Installation

#### Gradle

```gradle
repositories {
    mavenCentral()
}
dependencies {
    implementation("net.dv8tion:JDA:5.1.0")
}
```

#### Maven

```xml
<dependency>
    <groupId>net.dv8tion</groupId>
    <artifactId>JDA</artifactId>
    <version>5.1.0</version> 
</dependency>
```

</details>

<details>
<summary>spring-projects/spring-boot (org.springframework.boot:spring-boot-starter-parent)</summary>

### [`v3.3.3`](https://togithub.com/spring-projects/spring-boot/compare/v3.3.2...v3.3.3)

[Compare Source](https://togithub.com/spring-projects/spring-boot/compare/v3.3.2...v3.3.3)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 6am on saturday" in timezone Europe/Berlin, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/renovate/discussions) if that's undesired.

---

 - [ ] If you want to rebase/retry this PR, check this box

---

This PR was generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View the [repository job log](https://developer.mend.io/github/SvenKirschbaum/musikbot-client).
@UQuark
Copy link

UQuark commented Sep 3, 2024

Hello! I attempted to upgrade from version 5.0.2 to 5.1.0 today, but it caused an issue with receiving voice audio. The audio is either not coming at all or is just silence frames - I'm not sure on that one, further investigation is needed.
Reverting back to 5.0.2 resolved the problem. Is the new API currently in production?

My AudioReceiveHandler impl: https://pastebin.com/kxXUW2qG

@MinnDevelopment
Copy link
Member Author

Hello, the issue with voice receive has been fixed in a followup MR that hasn't made it into a new release yet. Please use the older version until then, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants