fix: avoid infinite recursion in case playabilityStatus NON_EMBEDDABLE (#54)

taraskondratiuk · Taras Kondratiuk · web-flow · commit 2d88c2948c8b · 2024-09-23T20:09:05.000+01:00
Co-authored-by: Taras Kondratiuk &lt;tkondratiuk@velitech.com&gt;
diff --git a/common/src/main/java/dev/lavalink/youtube/clients/skeleton/NonMusicClient.java b/common/src/main/java/dev/lavalink/youtube/clients/skeleton/NonMusicClient.java
@@ -117,7 +117,7 @@ protected JsonBrowser loadTrackInfoFromInnertube(@NotNull YoutubeAudioSourceMana
                     new RuntimeException("Non-embeddable video cannot be loaded by embedded client"));
             }
 
-            json = loadTrackInfoFromInnertube(source, httpInterface, videoId, status);
+            json = loadTrackInfoFromInnertube(source, httpInterface, videoId, playabilityStatus);
             getPlayabilityStatus(json.get("playabilityStatus"), true);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ protected JsonBrowser loadTrackInfoFromInnertube(@NotNull YoutubeAudioSourceMana`
`117`	`117`	`new RuntimeException("Non-embeddable video cannot be loaded by embedded client"));`
`118`	`118`	`}`
`119`	`119`
`120`		`- json = loadTrackInfoFromInnertube(source, httpInterface, videoId, status);`
	`120`	`+ json = loadTrackInfoFromInnertube(source, httpInterface, videoId, playabilityStatus);`
`121`	`121`	`getPlayabilityStatus(json.get("playabilityStatus"), true);`
`122`	`122`	`}`
`123`	`123`