Added Qobuz Source Manager #254

munishkhatri720 · 2025-03-07T20:41:53Z

No description provided.

topi314

auto format exist, use it

README.md

protocol/src/commonMain/kotlin/com/github/topi314/lavasrc/protocol/Config.kt

plugin/src/main/java/com/github/topi314/lavasrc/plugin/config/QobuzConfig.java

plugin/src/main/java/com/github/topi314/lavasrc/plugin/LavaSrcPlugin.java

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioTrack.java

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioSourceManager.java

topi314 · 2025-03-08T12:22:46Z

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioTrack.java

+            // throw new IllegalStateException("Failed to get track media URI");
+        }
+        if (!json.get("sample").isNull() && json.get("sample").asBoolean(true) == true) {
+            return getFallback();
+            // throw new IllegalStateException("Premium account required to play the whole
+            // track");
+        }


what is up with this commented code?

getFallback() uses a public api which allows to fetch stream urls from qobuz . Should i keep it or remove it ?

Without qobuz premium account the api only returns 30 second previews.

what kind of public api?

There is a music site, "music.wjhe.top," hosted by some Chinese people. That site supports playback from multiple sources, including Qobuz. The site uses the api to request streams that I added it here. Should I remove it?

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioTrack.java

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioSourceManager.java

README.md

topi314

cba to add a comment for each blank line. please fix.

README.md

main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioSourceManager.java

Copilot

Pull Request Overview

This pull request adds support for the Qobuz source by introducing a new Qobuz configuration, audio track implementation, and integrations into the plugin and documentation.

Added a new QobuzConfig class for managing Qobuz-related properties.
Implemented QobuzAudioTrack for handling Qobuz track media, including fallback mechanisms and signature generation.
Updated plugin configuration and documentation (README.md) to include Qobuz source manager details.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
plugin/src/main/java/com/github/topi314/lavasrc/plugin/config/QobuzConfig.java	Adds configuration support for Qobuz.
main/src/main/java/com/github/topi314/lavasrc/qobuz/QobuzAudioTrack.java	Implements Qobuz-specific audio track logic.
README.md	Updates documentation for Qobuz integration and usage.
plugin/src/main/java/com/github/topi314/lavasrc/plugin/LavaSrcPlugin.java	Integrates QobuzAudioSourceManager into the plugin.
plugin/src/main/java/com/github/topi314/lavasrc/plugin/config/SourcesConfig.java	Adds a flag to enable/disable Qobuz source support.

README.md

munishkhatri720 added 2 commits March 7, 2025 21:20

Added docs for qobuz and allowed updating userOauthToken via rest

d0cb0ce

improved docs and added supportd urls types

01be9bc

munishkhatri720 mentioned this pull request Mar 7, 2025

Added Qobuz Source Manager #244

Closed

topi314 requested changes Mar 8, 2025

View reviewed changes