diff --git a/app/gitea-client.js b/app/gitea-client.js index 6f6ce8e..f54fdd8 100644 --- a/app/gitea-client.js +++ b/app/gitea-client.js @@ -4,6 +4,18 @@ // 單一 HTTP 請求的逾時(毫秒)。避免 API 緩慢或掛起時容器永久卡死。 const REQUEST_TIMEOUT_MS = 30000 +/** + * 將回應內容整理成可安全寫入錯誤訊息的片段:移除控制字元(避免換行等造成的 log 注入)並限制長度。 + * @param {string} text 原始回應文字 + * @returns {string} 清理後、最長 200 字元的片段 + */ +function sanitizeBody(text) { + return (text || '') + .replace(/[\u0000-\u001F\u007F]+/g, ' ') + .trim() + .slice(0, 200) +} + /** * 與 Gitea REST API 溝通的輕量 HTTP 客戶端,負責帶上認證標頭、分頁讀取清單與發出刪除請求。 * 取代原 bash 版本以 `curl`/`jq` 進行的 API 操作。 @@ -22,14 +34,15 @@ export class GiteaClient { } /** - * 逐頁讀取分頁式清單 API(每頁以 `?page=N` 由 1 遞增),直到某頁回傳空陣列(或非陣列)為止, + * 逐頁讀取分頁式清單 API(每頁以 `?page=N` 由 1 遞增),直到某頁回傳空陣列為止, * 將所有頁面項目合併成單一陣列回傳。 * - * 注意:終止條件依賴「空陣列代表最後一頁」的假設;若 API 不以空陣列結尾則迴圈不會自然停止。 + * 在 `res.ok` 為真的前提下嚴格要求回應為 JSON 陣列:非 JSON、JSON 解析失敗或非陣列內容 + * 都會拋出帶 URL 上下文的錯誤,而非被誤判為最後一頁而靜默結束。 * * @param {string} baseUrl 不含 query string 的 API 位址(本方法會自行附加 `?page=N`) * @returns {Promise} 所有頁面合併後的項目陣列;無資料時為空陣列 - * @throws {Error} 任一頁回應 HTTP 非 2xx(`!res.ok`)時拋出,訊息含 URL 與狀態碼 + * @throws {Error} HTTP 非 2xx、回應非 JSON、JSON 無法解析或非陣列、或請求逾時(AbortError)時拋出 */ async fetchAllPages(baseUrl) { const all = [] @@ -43,27 +56,41 @@ export class GiteaClient { }) if (!res.ok) { - const body = await res.text().catch(() => '') + const body = sanitizeBody(await res.text().catch(() => '')) throw new Error( - `GET ${url} failed: HTTP ${res.status}${body ? ` - ${body.slice(0, 200)}` : ''}`, + `GET ${url} failed: HTTP ${res.status}${body ? ` - ${body}` : ''}`, ) } // 確認回應確實是 JSON,避免 API 回傳 HTML 錯誤頁時 res.json() 拋出難以理解的 SyntaxError。 const contentType = res.headers.get('content-type') || '' if (!contentType.includes('application/json')) { - const body = await res.text().catch(() => '') + const body = sanitizeBody(await res.text().catch(() => '')) throw new Error( - `GET ${url} returned non-JSON content-type "${contentType}": ${body.slice(0, 200)}`, + `GET ${url} returned non-JSON content-type "${contentType}"${body ? `: ${body}` : ''}`, ) } - const items = await res.json() - if (!Array.isArray(items) || items.length === 0) { + // 即使 content-type 正確,內容仍可能格式錯誤;明確攔截 SyntaxError 並補上 URL 上下文。 + let items + try { + items = await res.json() + } catch (error) { + throw new Error(`GET ${url} returned invalid JSON: ${error.message}`) + } + + // res.ok 為真時嚴格要求陣列:非陣列代表非預期回應(如錯誤物件),應報錯而非視為最後一頁。 + if (!Array.isArray(items)) { + throw new Error(`GET ${url} returned a non-array JSON payload`) + } + if (items.length === 0) { break } - all.push(...items) + // 逐一加入(不使用展開運算子),避免大量項目時觸發呼叫堆疊上限。 + for (const item of items) { + all.push(item) + } page += 1 }