input/tidal: invoke login handlers on second request

Fixes a freeze bug.

input/tidal: invoke login handlers on second request
cb87ed29 · Max Kellermann · 570c6765 · cb87ed29
Commit cb87ed29 authored Jan 21, 2018 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

TidalSessionManager.cxx src/input/plugins/TidalSessionManager.cxx +6 -2

No files found.
--- a/src/input/plugins/TidalSessionManager.cxx
+++ b/src/input/plugins/TidalSessionManager.cxx
@@ -47,7 +47,10 @@ TidalSessionManager::AddLoginHandler(TidalSessionHandler &h) noexcept
 	const bool was_empty = handlers.empty();
 	handlers.push_front(h);

-	if (was_empty && session.empty() && !login_request) {
+	if (!was_empty || login_request)
+		return;
+
+	if (session.empty()) {
 		// TODO: throttle login attempts?

 		std::string login_uri(base_url);
@@ -66,7 +69,8 @@ TidalSessionManager::AddLoginHandler(TidalSessionHandler &h) noexcept
 			ScheduleInvokeHandlers();
 			return;
 		}
-	}
+	} else
+		ScheduleInvokeHandlers();
 }

 void