-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
概要
downloadRunnerScriptAndUnZip 関数内の curl コマンドにエラーハンドリングとチェックサム検証を追加する必要があります。
問題点
- curlの
-fフラグが未指定: HTTPエラー(404, 500など)が発生してもダウンロードが成功したと判断されます - チェックサム検証なし: ダウンロードしたバイナリの整合性が検証されていないため、破損したファイルや改ざんされたファイルを実行する可能性があります
推奨される修正
curl -fsSL フラグを使用し、GitHubが提供するSHA256チェックサムを使用した検証を追加することを推奨します。
関連リンク
- PR: ファイル構造のリファクタリング + unitテストの追加 #507
- コメント: ファイル構造のリファクタリング + unitテストの追加 #507 (comment)
- リクエスト者: @mafreud
対象ファイル
openci-runner/firebase/functions/src/probot/github.ts の downloadRunnerScriptAndUnZip 関数
Metadata
Metadata
Assignees
Labels
No labels