91 lines
5.0 KiB
JSON
91 lines
5.0 KiB
JSON
[
|
|
{
|
|
"level": "critical",
|
|
"role": "Assassin",
|
|
"location": "Dockerfile:13",
|
|
"problem": "直接將網際網路的腳本經由 pipe 傳送給 shell 執行 (curl | sh) 極度危險，若該網址遭受劫持或供應商被駭，攻擊者可輕易於建置期間取得容器完整控制權。",
|
|
"suggestion": "應先將腳本下載至本地，透過 sha256sum 驗證其完整性後，再以 `sh` 執行。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "critical",
|
|
"role": "Assassin",
|
|
"location": "Dockerfile:16",
|
|
"problem": "從未經驗證的遠端 git 儲存庫安裝插件，若儲存庫內容遭竄改，將導致任意程式碼執行風險。",
|
|
"suggestion": "必須鎖定特定 commit hash (SHA) 來引用插件，確保所安裝內容符合預期且未遭篡改。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "critical",
|
|
"role": "Mage",
|
|
"location": "Dockerfile:17",
|
|
"problem": "在 Docker 構建階段直接執行 `codex plugin marketplace add` 並嘗試同時安裝與啟用，若網路不穩或遠端 Gitea 服務暫時不可用，構建會直接失敗，導致整個鏡像無法建立。",
|
|
"suggestion": "建議將插件安裝邏輯移至 `entrypoint.sh`，在執行時才確認環境並安裝，或者在 Dockerfile 中增加重試機制與錯誤捕獲。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "critical",
|
|
"role": "Maya",
|
|
"location": "Dockerfile:11",
|
|
"problem": "新增了 Codex CLI 的安裝與插件註冊過程，但沒有對應的測試來驗證安裝是否成功、插件是否正確載入。",
|
|
"suggestion": "應在測試套件中加入 Docker 映像檔的整合測試，驗證容器啟動後 `codex --version` 及插件列表是否存在。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "critical",
|
|
"role": "Assassin",
|
|
"location": "entrypoint.sh:20",
|
|
"problem": "使用了 `--dangerously-bypass-approvals-and-sandbox` 參數，這會導致安全沙盒失效。若輸入的 `$PROMPT` 或 `$MODEL` 可被外部使用者操縱，攻擊者可藉此在容器內執行任意指令。",
|
|
"suggestion": "移除此危險參數。必須落實沙盒隔離機制與人工審核流程，絕不可為了方便而犧牲安全性。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "critical",
|
|
"role": "Maya",
|
|
"location": "entrypoint.sh:18",
|
|
"problem": "修改了 `entrypoint.sh` 的核心邏輯以執行 `codex exec`，但缺乏驗證執行是否成功的測試案例（例如：當 `OAUTH` 錯誤、`MODEL` 無效或 `codex exec` 本身拋出錯誤時的處理）。",
|
|
"suggestion": "應為 `entrypoint.sh` 補上單元測試，模擬各種環境變數設定及 `codex exec` 的成功與失敗路徑，並驗證對應的退出碼。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "warning",
|
|
"role": "Assassin",
|
|
"location": "action.yaml:23",
|
|
"problem": "將包含認證資訊的 `OAUTH` 透過環境變數傳遞，這使得該敏感資訊在容器內的任何進程皆可透過環境變數（如 `/proc/self/environ`）輕易竊取。",
|
|
"suggestion": "應透過檔案掛載或將機密寫入至安全路徑後再讀取，避免直接透過環境變數傳遞敏感資訊。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "warning",
|
|
"role": "Leo",
|
|
"location": "Dockerfile:20",
|
|
"problem": "在建置映像檔時直接透過 `codex plugin marketplace add` 安裝插件，會增加映像檔建置的時間與複雜度，且若插件來源 Gitea 無法連線，映像檔就無法成功建置。",
|
|
"suggestion": "建議將常用的插件預先打包在映像檔中，或是透過環境變數配置插件路徑，減少建置期的網絡依賴。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "warning",
|
|
"role": "Mage",
|
|
"location": "entrypoint.sh:17",
|
|
"problem": "直接將 `OAUTH` 環境變數內容透過 `base64 -d` 寫入 `auth.json`，未校驗該變數是否為合法的 Base64 編碼。若輸入非 Base64 或損壞的字串，會產生損壞的認證檔案。",
|
|
"suggestion": "增加對 `OAUTH` 變數是否符合 Base64 格式的初步校驗，並在解碼失敗時明確報錯並終止。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "warning",
|
|
"role": "Maya",
|
|
"location": "entrypoint.sh:22",
|
|
"problem": "對於 `GITHUB_OUTPUT` 的處理有條件判斷，但缺乏測試驗證當此變數不存在時的行為，以及當存在時內容是否正確寫入。",
|
|
"suggestion": "應補測試案例驗證在 `GITHUB_OUTPUT` 設定與未設定的情況下，腳本是否皆能正常執行而不發生預期外的錯誤。",
|
|
"is_new": true
|
|
},
|
|
{
|
|
"level": "warning",
|
|
"role": "Rogue",
|
|
"location": "Dockerfile:17",
|
|
"problem": "每個 `RUN` 指令都會建立一個新的 Docker 映像層，在此處拆分為兩個獨立的 `RUN` 指令安裝插件，會產生不必要的額外層，增加了映像檔體積並浪費了建置與拉取映像檔的時間。",
|
|
"suggestion": "請將所有安裝插件的 `RUN` 指令透過 `&&` 合併為單一個 `RUN` 指令，以優化映像層數並提升效能。",
|
|
"is_new": true
|
|
}
|
|
]
|