Cache assets by server domain to prevent conflicts

Assets from different servers are now cached in separate subdirectories
based on the asset URL's host and path (e.g., direct.grave.wine/base/).
This prevents cache conflicts when connecting to different servers.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: OmniTroid

src/webcache.cpp

@@ -27,14 +27,41 @@ QString WebCache::cacheDir() const
   return get_base_path() + "webcache/";
 }
 
+QString WebCache::cacheSubdir() const
+{
+  QString assetUrl = ao_app->m_serverdata.get_asset_url();
+  if (assetUrl.isEmpty())
+  {
+    return QString();
+  }
+
+  // Extract host and path from asset URL (e.g., "https://direct.grave.wine/base/" -> "direct.grave.wine/base/")
+  QUrl url(assetUrl);
+  QString subdir = url.host() + url.path();
+
+  // Ensure it ends with /
+  if (!subdir.endsWith('/'))
+  {
+    subdir += '/';
+  }
+
+  return subdir;
+}
+
 QString WebCache::getCachedPath(const QString &relativePath) const
 {
   if (relativePath.isEmpty())
   {
     return QString();
   }
 
-  QString localPath = cacheDir() + relativePath;
+  QString subdir = cacheSubdir();
+  if (subdir.isEmpty())
+  {
+    return QString();
+  }
+
+  QString localPath = cacheDir() + subdir + relativePath;
 
   if (!file_exists(localPath))
   {
@@ -108,11 +135,18 @@ void WebCache::resolveOrDownload(const QString &relativePath, const QStringList
     return;
   }
 
+  // Get cache subdirectory for this server's asset URL
+  QString subdir = cacheSubdir();
+  if (subdir.isEmpty())
+  {
+    return;
+  }
+
   // Try each suffix
   for (const QString &suffix : suffixes)
   {
     QString fullPath = relativePath + suffix;
-    QString localPath = cacheDir() + fullPath;
+    QString localPath = cacheDir() + subdir + fullPath;
 
     // Skip if already cached and not expired
     if (file_exists(localPath) && !isExpired(localPath))

src/webcache.h

@@ -70,6 +70,12 @@ private Q_SLOTS:
   // Track failed downloads to avoid retrying them repeatedly
   QSet<QString> m_failed_downloads;
 
+  /**
+   * @brief Returns the cache subdirectory for the current server's asset URL.
+   * E.g., "https://direct.grave.wine/base/" -> "direct.grave.wine/base/"
+   */
+  QString cacheSubdir() const;
+
   /**
    * @brief Initiates an async download for the given remote URL.
    * @param remoteUrl The full URL to download from.