Finicky 4.0.0版本URL重写功能解析与修复

Finicky作为一个智能浏览器路由工具，其URL重写功能在4.0.0-alpha.3版本中出现了一个关键性bug。本文将深入分析这个问题的技术细节及其解决方案。

问题背景

在Finicky的配置中，开发者可以通过rewrite模块对URL进行预处理，然后再由handlers模块根据处理后的URL决定使用哪个浏览器打开。但在4.0.0-alpha.3版本中，虽然重写逻辑被执行，但最终处理的仍然是原始URL，而非重写后的URL。

技术分析

问题核心在于URL重写函数的递归调用逻辑存在缺陷。当重写函数返回一个新的URL对象时，系统错误地将原始URL而非重写后的URL传递给了后续处理流程。

具体表现为：

1. 重写模块确实执行了转换逻辑
2. 转换函数正确返回了新的URL对象
3. 但在递归调用rewriteUrl时，错误地将原始URL而非转换结果作为参数传递

问题复现

通过以下测试用例可以稳定复现该问题：

describe("rewriter + handlers", () => {
  const handlerConfig = {
    defaultBrowser: "Firefox",
    rewrite: [{
      match: (url) => true,
      url: (url) => new URL("https://example.com"),
    }],
    handlers: [{
      match: (url) => true,
      browser: "Firefox",
    }],
  };

  it("should use rewritten URL", () => {
    const result = openUrl("https://github.com/some-repo", handlerConfig);
    expect(result.browser.url).toBe("https://example.com");
  });
})

预期结果是处理后的URL应为"https://example.com"，但实际得到的是原始URL"https://github.com/some-repo"。

解决方案

修复方案主要调整了递归调用rewriteUrl时的参数传递逻辑。原先错误地将原始URL作为参数传递，修正后改为传递重写函数返回的结果URL。

该修复已包含在4.0.0-alpha.4版本中，确保了URL重写功能的正常运作。对于开发者而言，这意味着可以放心使用重写功能来实现各种URL转换需求，如：

• 深度链接处理
• 跟踪参数移除
• 域名重定向等

总结

URL重写是Finicky提供的一个强大功能，允许开发者在浏览器路由前对URL进行灵活处理。这次bug的发现和修复过程展示了开源社区协作的力量，也提醒我们在处理递归函数调用时需要特别注意参数传递的正确性。