Merge pull request 'release/v1.3.3' (#10) from release/v1.3.3 into main

Reviewed-on: #10

frontend/dist-electron/main.js

@@ -219,13 +219,9 @@ const scrapeAccountData = async (account) => {
                 }
             }
             catch (e) {
-                if (e instanceof scraper_1.SteamAuthError) {
+                if (e.message.includes('cookie') || e.message.includes('Sign In'))
                     account.authError = true;
             }
-                else {
-                    console.error(`[Scraper] Temporary error for ${account.personaName}: ${e.message}`);
-                }
-            }
         }
         if (backend && !account._id.startsWith('shared_')) {
             await backend.shareAccount(account);
@@ -263,16 +259,7 @@ const syncAccounts = async (isManual = false) => {
                 else {
                     const sDate = s.sessionUpdatedAt ? new Date(s.sessionUpdatedAt) : new Date(0);
                     const lDate = exists.sessionUpdatedAt ? new Date(exists.sessionUpdatedAt) : new Date(0);
-                    // 1. SENSITIVE DATA SYNC (Credentials)
+                    if (sDate > lDate) {
-                    const sSessionDate = s.sessionUpdatedAt ? new Date(s.sessionUpdatedAt) : new Date(0);
-                    const lSessionDate = exists.sessionUpdatedAt ? new Date(exists.sessionUpdatedAt) : new Date(0);
-                    const isLocalAccount = !exists._id.startsWith('shared_');
-                    const isLocalSessionHealthy = exists.steamLoginSecure && !exists.authError;
-                    // SMART OVERWRITE LOGIC:
-                    // - If it's a remote shared account: Newest wins.
-                    // - If it's a LOCAL account: Only overwrite if our local session is broken/missing.
-                    const shouldOverwriteCredentials = !isLocalAccount ? (sSessionDate > lSessionDate) : (!isLocalSessionHealthy && sSessionDate > lSessionDate);
-                    if (shouldOverwriteCredentials) {
                         if (s.loginName)
                             exists.loginName = s.loginName;
                         if (s.loginConfig)
@@ -285,7 +272,7 @@ const syncAccounts = async (isManual = false) => {
                         exists.sessionUpdatedAt = s.sessionUpdatedAt;
                         hasChanges = true;
                     }
-                    // 2. Metadata Sync (Pull) - Always "Newest Wins"
+                    // Metadata Sync (Pull)
                     const sMetaDate = s.lastMetadataCheck ? new Date(s.lastMetadataCheck) : new Date(0);
                     const lMetaDate = exists.lastBanCheck ? new Date(exists.lastBanCheck) : new Date(0);
                     if (sMetaDate > lMetaDate) {

frontend/dist-electron/services/scraper.js

@@ -36,17 +36,9 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.scrapeCooldown = exports.SteamAuthError = void 0;
+exports.scrapeCooldown = void 0;
 const axios_1 = __importDefault(require("axios"));
 const cheerio = __importStar(require("cheerio"));
-// Custom error to identify session death
-class SteamAuthError extends Error {
-    constructor(message) {
-        super(message);
-        this.name = "SteamAuthError";
-    }
-}
-exports.SteamAuthError = SteamAuthError;
 const scrapeCooldown = async (steamId, steamLoginSecure) => {
     const url = `https://steamcommunity.com/profiles/${steamId}/gcpd/730?tab=matchmaking`;
     try {
@@ -55,17 +47,13 @@ const scrapeCooldown = async (steamId, steamLoginSecure) => {
                 'Cookie': steamLoginSecure,
                 'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36'
             },
-            timeout: 10000,
+            timeout: 10000
-            validateStatus: (status) => status < 500 // Allow redirects to handle them manually
         });
-        // If Steam redirects us to the login page, the cookie is dead
-        if (response.data.includes('Sign In') || response.request.path.includes('/login')) {
-            throw new SteamAuthError('Invalid or expired steamLoginSecure cookie');
-        }
         const $ = cheerio.load(response.data);
-        if (!response.data.includes('Personal Game Data')) {
+        if (response.data.includes('Sign In') || !response.data.includes('Personal Game Data')) {
-            throw new SteamAuthError('Session invalid: Personal Game Data not accessible');
+            throw new Error('Invalid or expired steamLoginSecure cookie');
         }
+        // 1. Locate the specific table containing cooldown info
         let expirationDate = undefined;
         $('table').each((_, table) => {
             const headers = $(table).find('th').map((_, th) => $(th).text().trim()).get();
@@ -75,18 +63,25 @@ const scrapeCooldown = async (steamId, steamLoginSecure) => {
                 rows.each((_, row) => {
                     const dateText = $(row).find('td').eq(expirationIndex).text().trim();
                     if (dateText && dateText !== '') {
+                        // Steam uses 'GMT' which some JS engines don't parse well, replace with 'UTC'
                         const cleanDateText = dateText.replace(' GMT', ' UTC');
                         const parsed = new Date(cleanDateText);
                         if (!isNaN(parsed.getTime())) {
-                            if (!expirationDate || parsed > expirationDate)
+                            // We want the newest expiration date found
+                            if (!expirationDate || parsed > expirationDate) {
                                 expirationDate = parsed;
                             }
                         }
+                    }
                 });
             }
         });
         if (expirationDate && expirationDate.getTime() > Date.now()) {
-            return { isActive: true, expiresAt: expirationDate };
+            console.log(`[Scraper] Found active cooldown until: ${expirationDate.toISOString()}`);
+            return {
+                isActive: true,
+                expiresAt: expirationDate
+            };
         }
         const content = $('#personal_game_data_content').text();
         if (content.includes('Competitive Cooldown') || content.includes('Your account is currently')) {
@@ -95,10 +90,8 @@ const scrapeCooldown = async (steamId, steamLoginSecure) => {
         return { isActive: false };
     }
     catch (error) {
-        if (error instanceof SteamAuthError)
+        console.error(`[Scraper] Error for ${steamId}:`, error.message);
         throw error;
-        console.error(`[Scraper] Network/Internal Error for ${steamId}:`, error.message);
-        throw error; // Generic errors don't trigger re-auth
     }
 };
 exports.scrapeCooldown = scrapeCooldown;

frontend/dist-electron/services/steam-client.js

@@ -49,6 +49,10 @@ class SteamClientService {
             return null;
         return path_1.default.join(this.steamPath, 'config', 'config.vdf');
     }
+    /**
+     * Safe Atomic Write: Writes to a temp file and renames it.
+     * This prevents file corruption if the app crashes during write.
+     */
     safeWriteVdf(filePath, data) {
         const tempPath = `${filePath}.tmp_${Date.now()}`;
         const dir = path_1.default.dirname(filePath);
@@ -57,6 +61,7 @@ class SteamClientService {
                 fs_1.default.mkdirSync(dir, { recursive: true });
             const vdfContent = (0, simple_vdf_1.stringify)(data);
             fs_1.default.writeFileSync(tempPath, vdfContent, 'utf-8');
+            // Atomic rename
             fs_1.default.renameSync(tempPath, filePath);
         }
         catch (e) {
@@ -72,6 +77,7 @@ class SteamClientService {
         if (loginUsersPath && fs_1.default.existsSync(loginUsersPath)) {
             this.readLocalAccounts();
             chokidar_1.default.watch(loginUsersPath, { persistent: true, ignoreInitial: true }).on('change', () => {
+                console.log(`[SteamClient] loginusers.vdf changed, re-scanning...`);
                 this.readLocalAccounts();
             });
         }
@@ -83,7 +89,7 @@ class SteamClientService {
         try {
             const content = fs_1.default.readFileSync(filePath, 'utf-8');
             if (!content.trim())
-                return;
+                return; // Empty file
             const data = (0, simple_vdf_1.parse)(content);
             if (!data || !data.users)
                 return;
@@ -93,7 +99,8 @@ class SteamClientService {
                 if (!user || !user.AccountName)
                     continue;
                 accounts.push({
-                    steamId: steamId64, accountName: user.AccountName,
+                    steamId: steamId64,
+                    accountName: user.AccountName,
                     personaName: user.PersonaName || user.AccountName,
                     timestamp: parseInt(user.Timestamp) || 0
                 });
@@ -101,7 +108,9 @@ class SteamClientService {
             if (this.onAccountsChanged)
                 this.onAccountsChanged(accounts);
         }
-        catch (error) { }
+        catch (error) {
+            console.error('[SteamClient] Error parsing loginusers.vdf:', error);
+        }
     }
     extractAccountConfig(accountName) {
         const configPath = this.getConfigVdfPath();
@@ -114,6 +123,7 @@ class SteamClientService {
             return (accounts && accounts[accountName]) ? accounts[accountName] : null;
         }
         catch (e) {
+            console.error('[SteamClient] Failed to extract config.vdf data');
             return null;
         }
     }
@@ -121,7 +131,17 @@ class SteamClientService {
         const configPath = this.getConfigVdfPath();
         if (!configPath)
             return;
-        let data = { InstallConfigStore: { Software: { Valve: { Steam: { Accounts: {} } } } } };
+        let data = {
+            InstallConfigStore: {
+                Software: {
+                    Valve: {
+                        Steam: {
+                            Accounts: {}
+                        }
+                    }
+                }
+            }
+        };
         if (fs_1.default.existsSync(configPath)) {
             try {
                 const content = fs_1.default.readFileSync(configPath, 'utf-8');
@@ -131,23 +151,18 @@ class SteamClientService {
             }
             catch (e) { }
         }
-        const ensurePath = (obj, keys) => {
+        // Ensure safe nesting
-            let curr = obj;
+        if (!data.InstallConfigStore)
-            for (const key of keys) {
+            data.InstallConfigStore = {};
-                if (!curr[key] || typeof curr[key] !== 'object')
+        if (!data.InstallConfigStore.Software)
-                    curr[key] = {};
+            data.InstallConfigStore.Software = {};
-                curr = curr[key];
+        if (!data.InstallConfigStore.Software.Valve)
-            }
+            data.InstallConfigStore.Software.Valve = {};
-            return curr;
+        if (!data.InstallConfigStore.Software.Valve.Steam)
-        };
+            data.InstallConfigStore.Software.Valve.Steam = {};
-        const steamAccounts = ensurePath(data, ['InstallConfigStore', 'Software', 'Valve', 'Steam', 'Accounts']);
+        if (!data.InstallConfigStore.Software.Valve.Steam.Accounts)
-        // FAILPROOF: Force crucial flags that Steam uses to decide session validity
+            data.InstallConfigStore.Software.Valve.Steam.Accounts = {};
-        steamAccounts[accountName] = {
+        data.InstallConfigStore.Software.Valve.Steam.Accounts[accountName] = accountData;
-            ...accountData,
-            RememberPassword: "1",
-            AllowAutoLogin: "1",
-            Timestamp: Math.floor(Date.now() / 1000).toString()
-        };
         try {
             this.safeWriteVdf(configPath, data);
             console.log(`[SteamClient] Safely injected session for ${accountName}`);
@@ -205,7 +220,6 @@ class SteamClientService {
             }
             catch (e) { }
         }
-        // Injection of the actual authentication blob
         if (accountConfig && accountName) {
             this.injectAccountConfig(accountName, accountConfig);
         }
@@ -218,7 +232,11 @@ class SteamClientService {
             for (const regPath of regLocations) {
                 if (!fs_1.default.existsSync(path_1.default.dirname(regPath)))
                     continue;
-                let regData = { Registry: { HKCU: { Software: { Valve: { Steam: { AutoLoginUser: "", RememberPassword: "1", AlreadyLoggedIn: "1" } } } } } };
+                let regData = { Registry: { HKCU: { Software: { Valve: { Steam: {
+                                        AutoLoginUser: "",
+                                        RememberPassword: "1",
+                                        AlreadyLoggedIn: "1"
+                                    } } } } } };
                 if (fs_1.default.existsSync(regPath)) {
                     try {
                         const content = fs_1.default.readFileSync(regPath, 'utf-8');
@@ -228,6 +246,7 @@ class SteamClientService {
                     }
                     catch (e) { }
                 }
+                // Deep merge helper
                 const ensurePath = (obj, keys) => {
                     let curr = obj;
                     for (const key of keys) {

frontend/electron/main.ts

@@ -7,7 +7,7 @@ import axios from 'axios';
 import fs from 'fs';
 import { pathToFileURL } from 'url';
 import { fetchProfileData, scrapeBanStatus } from './services/steam-web';
-import { scrapeCooldown, SteamAuthError } from './services/scraper';
+import { scrapeCooldown } from './services/scraper';
 import { steamClient, LocalSteamAccount } from './services/steam-client';
 import { BackendService } from './services/backend';
 
@@ -247,11 +247,7 @@ const scrapeAccountData = async (account: Account) => {
           if (backend) await backend.pushCooldown(account.steamId, undefined, now.toISOString());
         }
       } catch (e: any) {
-        if (e instanceof SteamAuthError) {
+        if (e.message.includes('cookie') || e.message.includes('Sign In')) account.authError = true;
-          account.authError = true;
-        } else {
-          console.error(`[Scraper] Temporary error for ${account.personaName}: ${e.message}`);
-        }
       }
     }
     if (backend && !account._id.startsWith('shared_')) {
@@ -290,31 +286,15 @@ const syncAccounts = async (isManual = false) => {
         } else {
           const sDate = s.sessionUpdatedAt ? new Date(s.sessionUpdatedAt) : new Date(0);
           const lDate = exists.sessionUpdatedAt ? new Date(exists.sessionUpdatedAt) : new Date(0);
-          // 1. SENSITIVE DATA SYNC (Credentials)
+          if (sDate > lDate) {
-          const sSessionDate = s.sessionUpdatedAt ? new Date(s.sessionUpdatedAt) : new Date(0);
-          const lSessionDate = exists.sessionUpdatedAt ? new Date(exists.sessionUpdatedAt) : new Date(0);
-          
-          const isLocalAccount = !exists._id.startsWith('shared_');
-          const isLocalSessionHealthy = exists.steamLoginSecure && !exists.authError;
-
-          // SMART OVERWRITE LOGIC:
-          // - If it's a remote shared account: Newest wins.
-          // - If it's a LOCAL account: Only overwrite if our local session is broken/missing.
-          const shouldOverwriteCredentials = !isLocalAccount ? (sSessionDate > lSessionDate) : (!isLocalSessionHealthy && sSessionDate > lSessionDate);
-
-          if (shouldOverwriteCredentials) {
             if (s.loginName) exists.loginName = s.loginName;
             if (s.loginConfig) exists.loginConfig = s.loginConfig;
-            if (s.steamLoginSecure) {
+            if (s.steamLoginSecure) { exists.steamLoginSecure = s.steamLoginSecure; exists.autoCheckCooldown = true; exists.authError = false; }
-              exists.steamLoginSecure = s.steamLoginSecure;
-              exists.autoCheckCooldown = true;
-              exists.authError = false;
-            }
             exists.sessionUpdatedAt = s.sessionUpdatedAt;
             hasChanges = true;
           }
           
-          // 2. Metadata Sync (Pull) - Always "Newest Wins"
+          // Metadata Sync (Pull)
           const sMetaDate = s.lastMetadataCheck ? new Date(s.lastMetadataCheck) : new Date(0);
           const lMetaDate = exists.lastBanCheck ? new Date(exists.lastBanCheck) : new Date(0);
           if (sMetaDate > lMetaDate) {

frontend/electron/services/scraper.ts

@@ -6,14 +6,6 @@ export interface CooldownData {
   expiresAt?: Date;
 }
 
-// Custom error to identify session death
-export class SteamAuthError extends Error {
-  constructor(message: string) {
-    super(message);
-    this.name = "SteamAuthError";
-  }
-}
-
 export const scrapeCooldown = async (steamId: string, steamLoginSecure: string): Promise<CooldownData> => {
   const url = `https://steamcommunity.com/profiles/${steamId}/gcpd/730?tab=matchmaking`;
   
@@ -23,21 +15,16 @@ export const scrapeCooldown = async (steamId: string, steamLoginSecure: string):
         'Cookie': steamLoginSecure,
         'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36'
       },
-      timeout: 10000,
+      timeout: 10000
-      validateStatus: (status) => status < 500 // Allow redirects to handle them manually
     });
 
-    // If Steam redirects us to the login page, the cookie is dead
-    if (response.data.includes('Sign In') || response.request.path.includes('/login')) {
-      throw new SteamAuthError('Invalid or expired steamLoginSecure cookie');
-    }
-
     const $ = cheerio.load(response.data);
     
-    if (!response.data.includes('Personal Game Data')) {
+    if (response.data.includes('Sign In') || !response.data.includes('Personal Game Data')) {
-      throw new SteamAuthError('Session invalid: Personal Game Data not accessible');
+      throw new Error('Invalid or expired steamLoginSecure cookie');
     }
 
+    // 1. Locate the specific table containing cooldown info
     let expirationDate: Date | undefined = undefined;
 
     $('table').each((_, table) => {
@@ -49,10 +36,15 @@ export const scrapeCooldown = async (steamId: string, steamLoginSecure: string):
         rows.each((_, row) => {
           const dateText = $(row).find('td').eq(expirationIndex).text().trim();
           if (dateText && dateText !== '') {
+            // Steam uses 'GMT' which some JS engines don't parse well, replace with 'UTC'
             const cleanDateText = dateText.replace(' GMT', ' UTC');
             const parsed = new Date(cleanDateText);
+            
             if (!isNaN(parsed.getTime())) {
-              if (!expirationDate || parsed > (expirationDate as Date)) expirationDate = parsed;
+              // We want the newest expiration date found
+              if (!expirationDate || parsed > (expirationDate as Date)) {
+                expirationDate = parsed;
+              }
             }
           }
         });
@@ -60,7 +52,11 @@ export const scrapeCooldown = async (steamId: string, steamLoginSecure: string):
     });
 
     if (expirationDate && (expirationDate as Date).getTime() > Date.now()) {
-      return { isActive: true, expiresAt: expirationDate };
+      console.log(`[Scraper] Found active cooldown until: ${(expirationDate as Date).toISOString()}`);
+      return {
+        isActive: true,
+        expiresAt: expirationDate
+      };
     }
 
     const content = $('#personal_game_data_content').text();
@@ -70,8 +66,7 @@ export const scrapeCooldown = async (steamId: string, steamLoginSecure: string):
 
     return { isActive: false };
   } catch (error: any) {
-    if (error instanceof SteamAuthError) throw error;
+    console.error(`[Scraper] Error for ${steamId}:`, error.message);
-    console.error(`[Scraper] Network/Internal Error for ${steamId}:`, error.message);
+    throw error;
-    throw error; // Generic errors don't trigger re-auth
   }
 };

frontend/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "ultimate-ban-tracker-desktop",
-  "version": "1.3.3",
+  "version": "1.3.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "ultimate-ban-tracker-desktop",
-      "version": "1.3.3",
+      "version": "1.3.2",
       "license": "SEE LICENSE IN LICENSE",
       "dependencies": {
         "@emotion/react": "^11.14.0",

frontend/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ultimate-ban-tracker-desktop",
   "description": "Professional Steam Account Manager & Ban Tracker",
-  "version": "1.3.3",
+  "version": "1.3.2",
   "author": "Nils Pukropp <nils@narl.io>",
   "homepage": "https://narl.io",
   "license": "SEE LICENSE IN LICENSE",