Catching more empty URIs + log request body at TRACE level

Author: devgianlu

lib/src/main/java/xyz/gianlu/librespot/common/ProtoUtils.java

@@ -285,7 +285,7 @@ public static Player.ProvidedTrack convertToProvidedTrack(@Nullable ContextTrack
 
         Player.ProvidedTrack.Builder builder = Player.ProvidedTrack.newBuilder();
         builder.setProvider("context");
-        if (track.hasUri()) builder.setUri(track.getUri());
+        if (track.hasUri() && !track.getUri().isEmpty()) builder.setUri(track.getUri());
         if (track.hasUid()) builder.setUid(track.getUid());
 
         try {

lib/src/main/java/xyz/gianlu/librespot/dealer/DealerClient.java

@@ -94,7 +94,7 @@ private void handleRequest(@NotNull JsonObject obj) {
         String sender = payload.get("sent_by_device_id").getAsString();
 
         JsonObject command = payload.getAsJsonObject("command");
-        LOGGER.trace("Received request. {mid: {}, key: {}, pid: {}, sender: {}}", mid, key, pid, sender);
+        LOGGER.trace("Received request. {mid: {}, key: {}, pid: {}, sender: {}, command: {}}", mid, key, pid, sender, command);
 
         boolean interesting = false;
         synchronized (reqListeners) {

lib/src/main/java/xyz/gianlu/librespot/metadata/PlayableId.java

@@ -4,6 +4,7 @@
 import com.spotify.connectstate.Player;
 import com.spotify.metadata.Metadata;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 import xyz.gianlu.librespot.common.Utils;
 
 import java.util.Arrays;
@@ -50,9 +51,9 @@ static boolean canPlaySomething(@NotNull List<ContextTrack> tracks) {
         return false;
     }
 
-    @NotNull
+    @Nullable
     static PlayableId from(@NotNull Player.ProvidedTrack track) {
-        return fromUri(track.getUri());
+        return track.getUri().isEmpty() ? null : fromUri(track.getUri());
     }
 
     static boolean isSupported(@NotNull String uri) {