Repository navigation
fix: Node・Java・OSの信頼設定と関連依存を更新 - #570
Conversation
レビュー(テックリード/PM/PO)判定:コード差分の方向性は妥当。ただし実運用への適用経路に統合上の未解決事項があります(P1)。 テックリード:P1 — ABAffairs経由のデプロイにJDK拒否設定が届かないこのPRの
同様に、ABAffairsの 修正条件:ABAffairs側の両再利用ワークフロー参照と PM現在のPR説明は、CI完了と「マージ/デプロイ/プロセス再起動/疎通確認が残る」ことを分離しており適切です。一方、実行時の制御点である再利用ワークフローが旧版のままなので、このPRのCI成功をセキュリティリリース完了の根拠にしないことを求めます。運用導線の更新と適用先の実行イメージ・プロセスの確認を、完了判定へ含めてください。 POビルド時のTLS検証と実行コンテナ内の検証が揃った状態を受入条件にしてください。新規Dockerfile自体のOS/JDK対策、Go 1.26.10・Debian CA更新の下限チェック、Node 24.22.1への更新は、今回のコード差分では合理的です。既存イメージの再利用を禁止する必要はありませんが、対策が入った新しいイメージで稼働しているという証跡が必要です。 |
|
レビュー指摘に対応し、8941c2fda74fadf43680946a75beb2bf2412af17へ更新しました。
PM/POの受け入れ条件は継続します。PRの確認・マージ後、対策入りイメージのdigest、実プロセスのJava拒否設定・Node/CA版、再起動・疎通の証跡を揃えて最終判断します。コードとCIの修正だけで本番の対策完了とは扱いません。修正と証拠の再レビューをお願いします。 |
再レビュー結果(8941c2f/テックリード・PM・PO)前回指摘したP1は解消しました。コード/CI段階の再レビューは承認相当です。 テックリード
PM/PO前回の「CI成功≠実環境の対策完了」という区分がPR本文・対応コメントでも維持されており、適切です。今回の指摘を理由にコードのマージを止める必要はありません。 本番でのセキュリティ対策完了は別判定とし、対策込みの候補・固定コントローラーからの実配布、稼働イメージdigest/実行中のJDK拒否設定・CA版/再起動・疎通の証跡を照合してください。 判定:前回P1クローズ。PRのコードレビュー完了。本番受け入れは未実施のため別途残存。 |
実行環境とTLS信頼設定を更新し、eMudhra CA問題への対策をOS・Node.js・JDK・Goへ適用します。ビルド時と実行時の対策を揃える変更で、稼働環境への適用完了は別の受け入れ条件です。
変更
検証と到達点
残る受け入れ条件・レビュー観点
根拠: Node.js、Oracle、OpenJDK、Ubuntu、Amazon Linux、Go。