Fix rest routes

Author: devoxin

README.md

@@ -308,14 +308,19 @@ Body:
 ```
 
 Response:
+
+If the YouTube source is not enabled, or the `refreshToken` is invalid:
+`500 - Internal Server Error`
+
+Otherwise:
 `204 - No Content`
 
 ### `GET` `/youtube`
 
 Response:
 
 If the YouTube source is not enabled:
-`400 - Bad Request`
+`500 - Internal Server Error`
 
 Otherwise:
 ```json
@@ -324,8 +329,6 @@ Otherwise:
 }
 ```
 
-
-
 ## Migration from Lavaplayer's built-in YouTube source
 
 This client is intended as a direct replacement for Lavaplayer's built-in `YoutubeAudioSourceManager`,

plugin/build.gradle.kts

@@ -12,7 +12,7 @@ lavalinkPlugin {
     name = "youtube-plugin"
     path = "dev.lavalink.youtube.plugin"
     apiVersion = libs.versions.lavalink
-    serverVersion = "4.0.4"
+    serverVersion = "4.0.7"
     configurePublishing = false
 }
 

plugin/src/main/java/dev/lavalink/youtube/plugin/YoutubeRestHandler.java

@@ -2,16 +2,14 @@
 
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayerManager;
 import dev.lavalink.youtube.YoutubeAudioSourceManager;
+import dev.lavalink.youtube.plugin.rest.MinimalConfigResponse;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpStatus;
 import org.springframework.stereotype.Service;
 import org.springframework.web.bind.annotation.*;
 import org.springframework.web.server.ResponseStatusException;
 
-import java.util.Collections;
-import java.util.Map;
-
 @Service
 @RestController
 public class YoutubeRestHandler {
@@ -24,14 +22,14 @@ public YoutubeRestHandler(AudioPlayerManager playerManager) {
     }
 
     @GetMapping("/youtube")
-    public Map<String, String> getYoutubeConfig() {
+    public MinimalConfigResponse getYoutubeConfig() {
         YoutubeAudioSourceManager source = playerManager.source(YoutubeAudioSourceManager.class);
 
         if (source == null) {
             throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "The YouTube source manager is not registered.");
         }
 
-        return Collections.singletonMap("refreshToken", source.getOauth2RefreshToken());
+        return MinimalConfigResponse.from(source);
     }
 
     @PostMapping("/youtube")

plugin/src/main/java/dev/lavalink/youtube/plugin/rest/MinimalConfigResponse.java

@@ -0,0 +1,17 @@
+package dev.lavalink.youtube.plugin.rest;
+
+import dev.lavalink.youtube.YoutubeAudioSourceManager;
+import org.jetbrains.annotations.Nullable;
+
+public class MinimalConfigResponse {
+    @Nullable
+    public String refreshToken;
+
+    private MinimalConfigResponse(@Nullable String refreshToken) {
+        this.refreshToken = refreshToken;
+    }
+
+    public static MinimalConfigResponse from(YoutubeAudioSourceManager sourceManager) {
+        return new MinimalConfigResponse(sourceManager.getOauth2RefreshToken());
+    }
+}