移除 Baidu filesystem 对全局 DNR 规则的依赖，改为请求级禁用 cookie

Author: cyfung1031

packages/filesystem/baidu/baidu.test.ts

@@ -0,0 +1,28 @@
+import { describe, expect, it, vi, afterEach } from "vitest";
+import BaiduFileSystem from "./baidu";
+
+describe("BaiduFileSystem", () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it("request should omit credentials without using global DNR rules", async () => {
+    const fetchMock = vi.fn().mockResolvedValue({
+      json: async () => ({ errno: 0 }),
+    });
+    vi.stubGlobal("fetch", fetchMock);
+    const fs = new BaiduFileSystem("/apps", "token");
+
+    await expect(fs.request("https://pan.baidu.com/rest/2.0/xpan/file?method=list")).resolves.toEqual({
+      errno: 0,
+    });
+
+    expect(fetchMock).toHaveBeenCalledTimes(1);
+    expect(fetchMock).toHaveBeenCalledWith(
+      "https://pan.baidu.com/rest/2.0/xpan/file?method=list",
+      expect.objectContaining({
+        credentials: "omit",
+      })
+    );
+  });
+});

packages/filesystem/baidu/baidu.ts

@@ -59,29 +59,9 @@ export default class BaiduFileSystem implements FileSystem {
   async request(url: string, config?: RequestInit) {
     config = config || {};
     const headers = <Headers>config.headers || new Headers();
-    // 处理请求匿名不发送cookie
-    await chrome.declarativeNetRequest.updateDynamicRules({
-      removeRuleIds: [100],
-      addRules: [
-        {
-          id: 100,
-          action: {
-            type: "modifyHeaders",
-            responseHeaders: [
-              {
-                operation: "remove",
-                header: "cookie",
-              },
-            ],
-          },
-          condition: {
-            urlFilter: url,
-            resourceTypes: ["xmlhttprequest"],
-          },
-        },
-      ],
-    });
     config.headers = headers;
+    // 对百度网盘请求显式禁用 cookie，避免依赖全局 DNR 规则造成并发竞态
+    config.credentials = "omit";
     return fetch(url, config)
       .then((data) => data.json())
       .then(async (data) => {
@@ -99,11 +79,6 @@ export default class BaiduFileSystem implements FileSystem {
             });
         }
         return data;
-      })
-      .finally(() => {
-        chrome.declarativeNetRequest.updateDynamicRules({
-          removeRuleIds: [100],
-        });
       });
   }