From e331789a7b8e0e5c8ebbd50ecf1b211f52314fcf Mon Sep 17 00:00:00 2001
From: JustArchi <JustArchi@JustArchi.net>
Date: Fri, 16 Feb 2018 16:48:55 +0100
Subject: [PATCH] Avoid excessive session refreshes

I knew I forgot about something
---
 ArchiSteamFarm/ArchiWebHandler.cs | 33 ++++++++++++++++++++++---------
 1 file changed, 24 insertions(+), 9 deletions(-)

diff --git a/ArchiSteamFarm/ArchiWebHandler.cs b/ArchiSteamFarm/ArchiWebHandler.cs
index bba0c1508..18c449857 100644
--- a/ArchiSteamFarm/ArchiWebHandler.cs
+++ b/ArchiSteamFarm/ArchiWebHandler.cs
@@ -1284,7 +1284,17 @@ namespace ArchiSteamFarm {
 			return true;
 		}
 
-		private async Task<bool> RefreshSession() {
+		private async Task<bool> RefreshSession(string host) {
+			if (string.IsNullOrEmpty(host)) {
+				Bot.ArchiLogger.LogNullError(nameof(host));
+				return false;
+			}
+
+			string steamLogin = WebBrowser.CookieContainer.GetCookieValue(host, "steamLogin");
+			if (!string.IsNullOrEmpty(steamLogin)) {
+				return true;
+			}
+
 			if (!Bot.IsConnectedAndLoggedOn) {
 				return false;
 			}
@@ -1292,6 +1302,11 @@ namespace ArchiSteamFarm {
 			await SessionSemaphore.WaitAsync().ConfigureAwait(false);
 
 			try {
+				steamLogin = WebBrowser.CookieContainer.GetCookieValue(host, "steamLogin");
+				if (!string.IsNullOrEmpty(steamLogin)) {
+					return true;
+				}
+
 				if (!Bot.IsConnectedAndLoggedOn) {
 					return false;
 				}
@@ -1400,7 +1415,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlGetToHtmlDocumentRetryWithSession(host, request, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1442,7 +1457,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlGetToObjectRetryWithSession<T>(host, request, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1484,7 +1499,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlGetToXmlDocumentRetryWithSession(host, request, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1526,7 +1541,7 @@ namespace ArchiSteamFarm {
 				return true;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlHeadRetryWithSession(host, request, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1582,7 +1597,7 @@ namespace ArchiSteamFarm {
 				return true;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlPostRetryWithSession(host, request, data, referer, includeSessionInData, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1638,7 +1653,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlPostToHtmlDocumentRetryWithSession(host, request, data, referer, includeSessionInData, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1694,7 +1709,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlPostToObjectRetryWithSession<T>(host, url, data, referer, includeSessionInData, --maxTries).ConfigureAwait(false);
 			}
 
@@ -1753,7 +1768,7 @@ namespace ArchiSteamFarm {
 				return response.Content;
 			}
 
-			if (await RefreshSession().ConfigureAwait(false)) {
+			if (await RefreshSession(host).ConfigureAwait(false)) {
 				return await UrlPostToObjectRetryWithSession<T>(host, url, data, referer, includeSessionInData, --maxTries).ConfigureAwait(false);
 			}