Drop legacy WebAPI support

src/webui/webapplication.cpp

@@ -248,128 +248,19 @@ const Http::Environment &WebApplication::env() const
 
 void WebApplication::doProcessRequest()
 {
-    QString scope, action;
+    const QRegularExpressionMatch match = m_apiPathPattern.match(request().path);
-
+    if (!match.hasMatch()) {
-    const auto findAPICall = [&]() -> bool
+        sendWebUIFile();
-    {
-        QRegularExpressionMatch match = m_apiPathPattern.match(request().path);
-        if (!match.hasMatch()) return false;
-
-        action = match.captured(QLatin1String("action"));
-        scope = match.captured(QLatin1String("scope"));
-        return true;
-    };
-
-    const auto findLegacyAPICall = [&]() -> bool
-    {
-        QRegularExpressionMatch match = m_apiLegacyPathPattern.match(request().path);
-        if (!match.hasMatch()) return false;
-
-        struct APICompatInfo
-        {
-            QString scope;
-            QString action;
-            std::function<void ()> convertFunc;
-        };
-        const QMap<QString, APICompatInfo> APICompatMapping {
-            {"sync/maindata", {"sync", "maindata", nullptr}},
-            {"sync/torrent_peers", {"sync", "torrentPeers", nullptr}},
-
-            {"login", {"auth", "login", nullptr}},
-            {"logout", {"auth", "logout", nullptr}},
-
-            {"command/shutdown", {"app", "shutdown", nullptr}},
-            {"query/preferences", {"app", "preferences", nullptr}},
-            {"command/setPreferences", {"app", "setPreferences", nullptr}},
-            {"command/getSavePath", {"app", "defaultSavePath", nullptr}},
-
-            {"query/getLog", {"log", "main", nullptr}},
-            {"query/getPeerLog", {"log", "peers", nullptr}},
-
-            {"query/torrents", {"torrents", "info", nullptr}},
-            {"query/propertiesGeneral", {"torrents", "properties", nullptr}},
-            {"query/propertiesTrackers", {"torrents", "trackers", nullptr}},
-            {"query/propertiesWebSeeds", {"torrents", "webseeds", nullptr}},
-            {"query/propertiesFiles", {"torrents", "files", nullptr}},
-            {"query/getPieceHashes", {"torrents", "pieceHashes", nullptr}},
-            {"query/getPieceStates", {"torrents", "pieceStates", nullptr}},
-            {"command/resume", {"torrents", "resume", [this]() { m_params["hashes"] = m_params.take("hash"); }}},
-            {"command/pause", {"torrents", "pause", [this]() { m_params["hashes"] = m_params.take("hash"); }}},
-            {"command/recheck", {"torrents", "recheck", [this]() { m_params["hashes"] = m_params.take("hash"); }}},
-            {"command/resumeAll", {"torrents", "resume", [this]() { m_params["hashes"] = "all"; }}},
-            {"command/pauseAll", {"torrents", "pause", [this]() { m_params["hashes"] = "all"; }}},
-            {"command/rename", {"torrents", "rename", nullptr}},
-            {"command/download", {"torrents", "add", nullptr}},
-            {"command/upload", {"torrents", "add", nullptr}},
-            {"command/delete", {"torrents", "delete", [this]() { m_params["deleteFiles"] = "false"; }}},
-            {"command/deletePerm", {"torrents", "delete", [this]() { m_params["deleteFiles"] = "true"; }}},
-            {"command/addTrackers", {"torrents", "addTrackers", nullptr}},
-            {"command/setFilePrio", {"torrents", "filePrio", nullptr}},
-            {"command/setCategory", {"torrents", "setCategory", nullptr}},
-            {"command/addCategory", {"torrents", "createCategory", nullptr}},
-            {"command/removeCategories", {"torrents", "removeCategories", nullptr}},
-            {"command/getTorrentsUpLimit", {"torrents", "uploadLimit", nullptr}},
-            {"command/getTorrentsDlLimit", {"torrents", "downloadLimit", nullptr}},
-            {"command/setTorrentsUpLimit", {"torrents", "setUploadLimit", nullptr}},
-            {"command/setTorrentsDlLimit", {"torrents", "setDownloadLimit", nullptr}},
-            {"command/increasePrio", {"torrents", "increasePrio", nullptr}},
-            {"command/decreasePrio", {"torrents", "decreasePrio", nullptr}},
-            {"command/topPrio", {"torrents", "topPrio", nullptr}},
-            {"command/bottomPrio", {"torrents", "bottomPrio", nullptr}},
-            {"command/setLocation", {"torrents", "setLocation", nullptr}},
-            {"command/setAutoTMM", {"torrents", "setAutoManagement", nullptr}},
-            {"command/setSuperSeeding", {"torrents", "setSuperSeeding", nullptr}},
-            {"command/setForceStart", {"torrents", "setForceStart", nullptr}},
-            {"command/toggleSequentialDownload", {"torrents", "toggleSequentialDownload", nullptr}},
-            {"command/toggleFirstLastPiecePrio", {"torrents", "toggleFirstLastPiecePrio", nullptr}},
-
-            {"query/transferInfo", {"transfer", "info", nullptr}},
-            {"command/alternativeSpeedLimitsEnabled", {"transfer", "speedLimitsMode", nullptr}},
-            {"command/toggleAlternativeSpeedLimits", {"transfer", "toggleSpeedLimitsMode", nullptr}},
-            {"command/getGlobalUpLimit", {"transfer", "uploadLimit", nullptr}},
-            {"command/getGlobalDlLimit", {"transfer", "downloadLimit", nullptr}},
-            {"command/setGlobalUpLimit", {"transfer", "setUploadLimit", nullptr}},
-            {"command/setGlobalDlLimit", {"transfer", "setDownloadLimit", nullptr}}
-        };
-
-        const QString legacyAction {match.captured(QLatin1String("action"))};
-        const APICompatInfo compatInfo = APICompatMapping.value(legacyAction);
-
-        scope = compatInfo.scope;
-        action = compatInfo.action;
-        if (compatInfo.convertFunc)
-            compatInfo.convertFunc();
-
-        const QString hash {match.captured(QLatin1String("hash"))};
-        if (!hash.isEmpty())
-            m_params[QLatin1String("hash")] = hash;
-
-        return true;
-    };
-
-    if (!findAPICall())
-        findLegacyAPICall();
-
-    APIController *controller = m_apiControllers.value(scope);
-    if (!controller) {
-        if (request().path == QLatin1String("/version/api")) {
-            print(QString::number(COMPAT_API_VERSION), Http::CONTENT_TYPE_TXT);
         return;
     }
 
-        if (request().path == QLatin1String("/version/api_min")) {
+    const QString action = match.captured(QLatin1String("action"));
-            print(QString::number(COMPAT_API_VERSION_MIN), Http::CONTENT_TYPE_TXT);
+    const QString scope = match.captured(QLatin1String("scope"));
-            return;
-        }
 
-        if (request().path == QLatin1String("/version/qbittorrent")) {
+    APIController *controller = m_apiControllers.value(scope);
-            print(QString(QBT_VERSION), Http::CONTENT_TYPE_TXT);
+    if (!controller)
-            return;
+        throw NotFoundHTTPError();
-        }
 
-        sendWebUIFile();
-    }
-    else {
     if (!session() && !isPublicAPI(scope, action))
         throw ForbiddenHTTPError();
 
@@ -408,7 +299,6 @@ void WebApplication::doProcessRequest()
             Q_ASSERT(false);
         }
     }
-    }
 }
 
 void WebApplication::configure()

src/webui/webapplication.h

@@ -44,8 +44,6 @@
 #include "base/utils/version.h"
 
 constexpr Utils::Version<int, 3, 2> API_VERSION {2, 2, 0};
-constexpr int COMPAT_API_VERSION = 23;
-constexpr int COMPAT_API_VERSION_MIN = 23;
 
 class APIController;
 class WebApplication;
@@ -131,7 +129,6 @@ private:
     QMap<QString, QString> m_params;
 
     const QRegularExpression m_apiPathPattern {(QLatin1String("^/api/v2/(?<scope>[A-Za-z_][A-Za-z_0-9]*)/(?<action>[A-Za-z_][A-Za-z_0-9]*)$"))};
-    const QRegularExpression m_apiLegacyPathPattern {QLatin1String("^/(?<action>((sync|command|query)/[A-Za-z_][A-Za-z_0-9]*|login|logout))(/(?<hash>[^/]+))?$")};
 
     QHash<QString, APIController *> m_apiControllers;
     QSet<QString> m_publicAPIs;