Skip to content

[v1.3] fix #1235#1238

Open
cyfung1031 wants to merge 1 commit intoscriptscat:release/v1.3from
cyfung1031:fix-link-decoding004
Open

[v1.3] fix #1235#1238
cyfung1031 wants to merge 1 commit intoscriptscat:release/v1.3from
cyfung1031:fix-link-decoding004

Conversation

@cyfung1031
Copy link
Collaborator

@cyfung1031 cyfung1031 commented Feb 13, 2026

应作者 @CodFrm 要求不写通用代码
只做单一修正

prettyUrl 不改
不加入 toEncodedUrl

只做紧急处理

@CodFrm
Copy link
Member

CodFrm commented Feb 13, 2026

不需要针对GreasyFork、SleazyFork特殊处理呀,如果其他平台,也搞个有问题的url不可能也去特殊处理啊,这个bug产生的原因是 searchParams 取的url参数有问题,换成location保持原本的值去fetch脚本数据就可以了,prettyUrl #1236 的看起来没啥问题

const url = searchParams.get("url");

@cyfung1031
Copy link
Collaborator Author

不需要针对GreasyFork、SleazyFork特殊处理呀,这个bug产生的原因是 searchParams 取的url参数有问题

  1. chrome declarative rule 的 regexSubstitution 不会进行 encodeURI / encodeURIComponent 等预处理。chrome会把解码的原始url放在 \1
  2. 在install页面看到的是一般 url 的 encodeURI 处理过的东西 (这里假设了 chrome 现时版本 143. 这PR写的是通用方式针对 Firefox 及未来Chrome)
  3. 我在这加了处理能让 install页面取得原本的原始网址
  4. GF和SF的档案名是用encodeURIComponent. 所以不能直接 new URL.

之前的 toEncodedURL是把原始网址正确编码
但你不喜欢,我就不再这里纠缠了
只需理GF和SF

prettyUrl 更不是我写的。我没有责任让它完善

@CodFrm
Copy link
Member

CodFrm commented Feb 13, 2026

怎么感觉搞反了呢,我是觉得不需要特殊处理,prettyUrl可以(另外这个是你写的#993,当然这个无所谓啦)

@cyfung1031
Copy link
Collaborator Author

(另外这个是你写的https://github.com/scriptscat/scriptcat/pull/993,当然这个无所谓啦)

呀。我失忆了。对不起
完全没印象

怎么感觉搞反了呢,我是觉得不需要特殊处理

对于问题,我提出了例子,提出了解决用的代码,提出了见解
也接受了你在 1236 的指责。bug 修补以外的都拿走

你是作者,接不接受在于你

@cyfung1031 cyfung1031 added the hotfix 需要尽快更新到扩展商店 label Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hotfix 需要尽快更新到扩展商店

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants