qq-bot-roost 0.8.10 → 0.8.12

package/README.md

@@ -10,7 +10,7 @@
 <img src="https://img.shields.io/badge/dependencies-0-brightgreen" alt="零依赖">
 <img src="https://img.shields.io/badge/tests-600%2B%20passed-brightgreen" alt="测试 600+ 项">
 <img src="https://img.shields.io/badge/plugins-JS%20%7C%20Python%20%7C%20Go%20%7C%20any-blue" alt="多语言插件">
-<img src="https://img.shields.io/badge/version-0.8.9-blueviolet" alt="版本 0.8.9">
+<img src="https://img.shields.io/badge/version-0.8.12-blueviolet" alt="版本 0.8.12">
 <img src="https://img.shields.io/badge/license-Non--Commercial-red" alt="禁止商用">
 <a href="https://github.com/tdcq9594/qq-bot-roost/actions"><img src="https://img.shields.io/github/actions/workflow/status/tdcq9594/qq-bot-roost/ci.yml?branch=master" alt="CI"></a>
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qq-bot-roost",
-  "version": "0.8.10",
+  "version": "0.8.12",
   "type": "module",
   "description": "栖 Roost：零依赖的 QQ 官方机器人框架，填 AppID/AppSecret 即连，插件可写任意语言",
   "license": "SEE LICENSE IN LICENSE",

package/src/plugin/deps.js

@@ -239,6 +239,42 @@ function venvPython(venvDir) {
     : path.join(venvDir, 'bin', 'python');
 }
 
+/**
+ * requirements.txt 单行的白名单。
+ *
+ * 只接受「包名 + 版本约束」（可带 extras、可带环境标记）这一种形态，其它一律拒。
+ * 不能用黑名单——pip 的选项缩写太多（`-i` = `--index-url`、`-c` = 约束文件、
+ * `-r` = 嵌套 requirement、`--no-index`、`--pre`…），黑名单列不完，
+ * 漏一个就是一条供应链后门：插件作者能把 pip 指到任意源、拉任意代码。
+ *
+ * 环境标记里也不许出现 `--`，否则 `pkg; x --index-url evil` 会绕过这条意图。
+ */
+const REQUIREMENT_RE =
+  /^[A-Za-z0-9][A-Za-z0-9._-]*(?:\[[A-Za-z0-9._,-]+\])?(?:\s*(?:==|>=|<=|~=|!=|===|>|<)\s*[^\s,]+(?:\s*,\s*[^\s,]+)*)?(?:\s*;\s*(?:(?!--)[^\n])+)?$/;
+
+/**
+ * pip 自己的注释规则（见 pip._internal.req.req_file.COMMENT_RE）：(^|\s+)#.*$
+ * 套白名单前必须按同样规则剥掉，否则 `requests>=2.31  # 固定下限` 这种合法写法
+ * 会被误杀——这里是 throw，误杀等于整个插件装不上。
+ */
+const stripReqComment = (l) => l.replace(/(^|\s+)#.*$/, '').trim();
+
+/**
+ * 校验 requirements.txt 全文。
+ * 抽成独立函数是为了能直接测：白名单一旦被改坏（拒了合法行 / 放了危险行），
+ * 靠构造假 python 走 ensurePluginDeps 很难测全。
+ *
+ * @param {string} requirements 文件全文
+ * @returns {{ok: true} | {ok: false, line: string}} line 是第一条不安全的行（已剥注释）
+ */
+export function checkRequirements(requirements) {
+  const bad = String(requirements ?? '')
+    .split(/\r?\n/)
+    .map((l) => stripReqComment(l.trim()))
+    .find((l) => l !== '' && !REQUIREMENT_RE.test(l));
+  return bad === undefined ? { ok: true } : { ok: false, line: bad };
+}
+
 /**
  * 确保插件的依赖装好。
  *
@@ -311,27 +347,14 @@ export async function ensurePluginDeps({ dir, manifest = {}, log = null, enabled
       /**
        * requirements.txt 里能写 `--index-url`、`-e git+...`、直接写 URL——
        * 等于让插件作者把 pip 指到任意源、拉任意代码。
-       *
-       * 这里用**白名单**（fail-closed）：只接受「包名 + 版本约束」这一种形态，
-       * 其它一律拒。不能用黑名单——pip 的选项缩写太多（`-i` = `--index-url`、
-       * `-c` = 约束文件、`-r` = 嵌套 requirement、`--no-index`、`--pre`…），
-       * 黑名单列不完，漏一个就是一条供应链后门。
-       *
-       * 合法行只匹配：包名（字母数字 + `-_.`）后面可选地跟版本约束
-       * （`==` / `>=` / `~=` / `<=` / `!=` / `>` / `<` 等，或逗号分隔的多个）。
-       * 环境标记（`; python_version < "3.8"`）一并放行，它不含网络指令。
+       * 白名单的细节和理由见上面的 checkRequirements()。
        */
-      const badLine = requirements
-        .split(/\r?\n/)
-        .map((l) => l.trim())
-        .find((l) => {
-          if (!l || l.startsWith('#')) return false; // 空行 / 注释放行
-          return !/^[A-Za-z0-9][A-Za-z0-9._-]*(?:\[[A-Za-z0-9._,-]+\])?(?:\s*(?:==|>=|<=|~=|!=|===|>|<)\s*[^\s,]+(?:\s*,\s*[^\s,]+)*)?(?:\s*;\s*[^\n]+)?$/.test(l);
-        });
-      if (badLine) {
+      const rq = checkRequirements(requirements);
+      if (!rq.ok) {
         throw new Error(
-          `requirements.txt 里有不安全的行，已拒绝安装：${badLine}\n` +
-            '只允许「包名 + 版本约束」（如 requests>=2.31），不接受 -e / --index-url / 直接 URL / pip 选项。',
+          `requirements.txt 里有不安全的行，已拒绝安装：${rq.line}\n` +
+            '只允许「包名 + 版本约束」（如 requests>=2.31），不接受 -e / --index-url / 直接 URL / pip 选项。' +
+            '行尾注释可以写（`# xxx`），但不支持反斜杠续行。',
         );
       }
 

package/src/web/server.js

@@ -111,6 +111,57 @@ const CSP_PAGE = [
   "connect-src http: https:",
 ].join('; ');
 
+/**
+ * 插件页的「自动带令牌」补丁 —— 注入到插件页 HTML 最前面。
+ *
+ * **为什么需要（0.8.12 修的回归）**：iframe 地址上带的凭证以前是面板密钥
+ * `?key=`，插件页面自己把它转发给接口。0.8.9 起换成了一次性页面令牌
+ * `?pt=`，但**已有插件的页面还在转发 `key`** —— 于是它们发出的请求一个
+ * 凭证都没带；iframe 又是 opaque origin（`Origin: null`），`isSameOrigin()`
+ * 判跨站 → 403，页面永远停在「加载中…」，控制台只看到 `Failed to load
+ * resource: 403`。dice 的战绩页就是这么挂的。
+ *
+ * 不能指望每个插件作者都回来改这一行（市场里的插件都在外部仓库），
+ * 所以框架替页面把 `pt` 补上。
+ *
+ * **安全边界没变**：
+ * - 只补发给**面板自己**（目标 origin 必须等于文档基准的 origin），
+ *   第三方站点拿不到令牌；
+ * - 能碰哪些路径仍由 `pageTokenAllows()` 决定，拿着令牌也调不了
+ *   /api/config、/api/logs、/api/bots。
+ *
+ * @param {string} pt 本次请求的页面令牌
+ * @returns {string} 一段 `<script>`，无令牌时返回空串
+ */
+export const pageTokenPatch = (pt) => {
+  const token = String(pt ?? '').replace(/[^A-Za-z0-9]/g, '');
+  if (!token) return '';
+  return (
+    '<script>(function(){var PT=' +
+    JSON.stringify(token) +
+    ',BASE=document.baseURI||location.href,SELF="";' +
+    'try{SELF=new URL(BASE).origin}catch(e){}' +
+    // 只给面板自己的地址补令牌：第三方 origin 原样发出，绝不外泄
+    'function add(u){try{var n=new URL(u,BASE);' +
+    'if(SELF&&n.origin!==SELF)return u;' +
+    'if(n.searchParams.get("pt"))return n.href;' +
+    'n.searchParams.set("pt",PT);return n.href}catch(e){return u}}' +
+    'var f=window.fetch;if(f)window.fetch=function(i,o){try{' +
+    'if(typeof i==="string")i=add(i);' +
+    'else if(i instanceof URL)i=new URL(add(i.href));' +
+    'else if(typeof Request!=="undefined"&&i instanceof Request){var h=add(i.url);' +
+    'if(h!==i.url)i=new Request(h,i)}}catch(e){}return f.call(window,i,o)};' +
+    'var X=window.XMLHttpRequest;' +
+    'if(X&&X.prototype&&X.prototype.open){var O=X.prototype.open;' +
+    'X.prototype.open=function(m,u){if(typeof u==="string"){' +
+    'try{arguments[1]=add(u)}catch(e){}}return O.apply(this,arguments)}};' +
+    'var E=window.EventSource;' +
+    'if(E){var W=function(u,c){return new E(add(String(u)),c)};W.prototype=E.prototype;' +
+    'try{W.CONNECTING=E.CONNECTING;W.OPEN=E.OPEN;W.CLOSED=E.CLOSED}catch(e){}' +
+    'window.EventSource=W}})();<\/script>'
+  );
+};
+
 const MIME = {
   '.html': 'text/html; charset=utf-8',
   '.css': 'text/css; charset=utf-8',
@@ -1637,9 +1688,15 @@ export async function startWebServer({
            * 所以有 doctype 就插在它后面。
            */
           const style = PAGE_THEME[theme];
+          /**
+           * 自动带令牌的补丁必须比页面自己的脚本**先**执行，所以跟主题样式
+           * 一起插在最前面（doctype 之后 / 没有 doctype 就文档开头）。
+           */
+          const patch = pageTokenPatch(url.searchParams.get('pt') || ptPage?.token || '');
+          const prefix = patch + style;
           const html = /^<!doctype/i.test(p.html.trimStart())
-            ? p.html.replace(/^(\s*<!doctype[^>]*>)/i, `$1${style}`)
-            : style + p.html;
+            ? p.html.replace(/^(\s*<!doctype[^>]*>)/i, `$1${prefix}`)
+            : prefix + p.html;
           return res.end(html);
         }