- Send data to clients only if they are authenticated.

- Send songs of the active playlist if the active playlist changes.

src/networkremote/outgoingdatacreator.cpp

@@ -125,6 +125,10 @@ void OutgoingDataCreator::SendAllPlaylists() {
 }
 
 void OutgoingDataCreator::ActiveChanged(Playlist* playlist) {
+  // Send the tracks of the active playlist
+  SendPlaylistSongs(playlist->id());
+
+  // Send the changed message after sending the playlist songs
   pb::remote::Message msg;
   msg.set_type(pb::remote::ACTIVE_PLAYLIST_CHANGED);
   msg.mutable_response_active_changed()->set_id(playlist->id());

src/networkremote/remoteclient.cpp

@@ -131,20 +131,22 @@ void RemoteClient::DisconnectClient(pb::remote::ReasonDisconnect reason) {
 }
 
 void RemoteClient::SendData(pb::remote::Message *msg) {
-  // Serialize the message
-  std::string data = msg->SerializeAsString();
+  if (authenticated_) {
+    // Serialize the message
+    std::string data = msg->SerializeAsString();
 
-  // Check if we are still connected
-  if (client_->state() == QTcpSocket::ConnectedState) {
-    // write the length of the data first
-    QDataStream s(client_);
-    s << qint32(data.length());
-    s.writeRawData(data.data(), data.length());
+    // Check if we are still connected
+    if (client_->state() == QTcpSocket::ConnectedState) {
+      // write the length of the data first
+      QDataStream s(client_);
+      s << qint32(data.length());
+      s.writeRawData(data.data(), data.length());
 
-    // Do NOT flush data here! If the client is already disconnected, it
-    // causes a SIGPIPE termination!!!
-  } else {
-    client_->close();
+      // Do NOT flush data here! If the client is already disconnected, it
+      // causes a SIGPIPE termination!!!
+    } else {
+      client_->close();
+    }
   }
 }