修正 fetchScriptBody 的 Accept 设定,避免触发 Error 406#1306
修正 fetchScriptBody 的 Accept 设定,避免触发 Error 406#1306cyfung1031 wants to merge 2 commits intoscriptscat:mainfrom
Conversation
src/pages/install/App.tsx
Outdated
| "Cache-Control": "no-cache", | ||
| Accept: "text/javascript,application/javascript,text/plain,application/octet-stream,application/force-download", | ||
| /* 不指定 application/octet-stream 和 application/force-download 避免触发伺服器端 Error 406 */ | ||
| Accept: "text/javascript, application/javascript, */*", // prefer JavaScript, but anything is acceptable |
There was a problem hiding this comment.
这个还好吧。之前没 */*
加了 */* 后就不会让 Server报错
只是 javascript 优先
src/pages/install/App.tsx
Outdated
| const contentType = response.headers.get("content-type"); | ||
|
|
||
| if (contentType) { | ||
| // 不接受非 JavaScript文本 的回应 |
There was a problem hiding this comment.
避免恶意连结。如果类型不对,就不下载了
There was a problem hiding this comment.
这个包括网址不正确的跳页。会直接不下载而不是下载后再解析
|
@CodFrm 你再看看吧。我认为这些测试都是必须的。因为不知道用户会点了什么连结 |
去了吧,我们也不清楚服务端会不会按照所规定的来执行,也会导致错误;反正最后会实际的解析脚本代码,来确定脚本是否正确 |
之前 Accept 没加 |
如果不加这些,是不是都不会有这个问题 |
Code reviewFound 1 issue:
scriptcat/src/pages/install/App.tsx Lines 93 to 99 in b3506a2 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
Checklist / 检查清单
Description / 描述
修正 fetchScriptBody 的 Accept 设定,避免触发 Error 406
(see #1271)
测试: https://openuserjs.org/scripts/alexchen/eHunter
Screenshots / 截图