Tangled.org pull requests break (?ref is stripped)

未关闭 适合新手
  • 哪里不对？
#353 0 条评论 2 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
68/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
javascript
领域
tooling

调研方向

找到与 (?:%3F)?ref_? 匹配的规则条目，并检查它如何应用于日志中显示的 api.tangled.org 请求。先阅读周边规则，以及仓库中有关验证规则更改的任何指南。完成标准是 ClearURLs 不再从受影响的请求中移除 ref，同时项目的检查仍覆盖该规则现有的行为。

由索引模型根据 Issue 内容生成。

描述

With ClearURLs active, I cannot merge PRs on tangled.org.
The issue is caused by the ?ref= parameter getting stripped.

Confirmed ClearURLs is causing the issue.
https://discord.com/channels/1361963801993285692/1361963802848919676/1565846122390364401

Image

ClearURLsLogExport.json

{
    "log": [
        {
            "before": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote&ref=refs%2Fheads%2Fmain",
            "after": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote",
            "rule": "(?:%3F)?ref_?",
            "timestamp": 1793397940444
        },
        {
            "before": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote&ref=refs%2Fheads%2Fmain",
            "after": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote",
            "rule": "(?:%3F)?ref_?",
            "timestamp": 1793397940444
        },
        {
            "before": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote&ref=refs%2Fheads%2Fdocs%2Fcar-voice-button",
            "after": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote",
            "rule": "(?:%3F)?ref_?",
            "timestamp": 1793397940444
        },
        {
            "before": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote&ref=refs%2Fheads%2Fmain",
            "after": "https://api.tangled.org/xrpc/org.tangled.temp.git.getRef?repo=did%3Aplc%3Axln6y55ck4cdkuna5gfixote",
            "rule": "(?:%3F)?ref_?",
            "timestamp": 1793397943816
        }
    ]
}
主要语言
JavaScript
星标
253
派生
85
30 天内没有已合并 PR

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

ClearURLs/Rules 的其他 Issue

查看 ClearURLs/Rules 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。