Skip to content

依存パッケージの脆弱性を修正し、保全実績の登録を保全担当者に限り、定期チェックを入れる - #29

Closed
msmsrep wants to merge 3 commits into
mainfrom
dev
Closed

msmsrep wants to merge 3 commits into
mainfrom
dev

Conversation

@msmsrep

@msmsrep msmsrep commented Oct 22, 2026

Copy link
Copy Markdown
Owner

概要

脆弱性点検の指摘のうち、要対応の2件を直し、再発を防ぐための定期チェックを追加する。

変更

1. 既知の脆弱性があるパッケージを修正版へ上げる(9a4b300)

  • Microsoft.AspNetCore.Components.WebAssembly.Server 10.0.10(High:GHSA-g27p-qf6j-c96c・GHSA-qjpx-jqm5-h2c4)ほか、Microsoft.* の 10.0.10 をすべて 10.0.13 に揃えた
  • System.Security.Cryptography.Pkcs 9.0.4(SqlClient 6.1.1 経由。High:GHSA-846f-f764-76mx ほか)は Infrastructure から 10.0.13 を直接参照して上書きした
  • Microsoft.OpenApi を 2.12.2 へ上げた(AspNetCore.OpenApi 10.0.13 の要求。上げないと NU1605 でビルドエラー)
  • dotnet-ef ツールを 10.0.13 に揃えた

2. 保全実績の登録を保全担当者に限る(9de7a7c、Spec.md 改訂144、E-40-30-01)

  • 単票API POST api/maintenance-orders/{id}/record と実績CSV maintenance-records が全ロールに開いていた。物流・品質の担当者でも、在庫の引落し・保全指示の完了・設備の復帰・治工具の寿命リセットができた
  • API・CSVとも MesRoleGroups.MaintenanceManage で絞り、画面の実績登録フォームも同じ定数で隠した
  • 突発の保全依頼の起票は従来どおり全ロールに開いている(Spec.md 7.4)

3. 依存の更新確認と脆弱性チェックを定期実行する(547d5e1)

  • Dependabot(nuget・github-actions):毎週月曜 09:00(JST)に dev 向けの PR を作る。担当者は msmsrep
  • Vulnerability ワークフロー:push・PR・週1回の定期実行で、推移的な依存も含めて既知の脆弱性を一覧にし、見つかったら失敗させる
  • Dependabot と定期実行は、main にマージされてから有効になる

確認

  • ビルド成功・新規の警告なし、テスト全件成功(Api 282件・Client.Web 229件)、scripts/I18nCheck.ps1 で取りこぼしなし
  • dotnet list package --vulnerable --include-transitive で脆弱なパッケージが0件
  • 画面(新しいDB):作業者には実績登録フォームが出ず、保全担当者には出て、登録すると指示が「完了」になる
  • 脆弱性チェックのスクリプトを手元で実行:修正前のコミットでは4件を挙げて失敗し、現在の状態では成功

🤖 Generated with Claude Code

msmsrep and others added 3 commits October 23, 2026 07:09
dotnet list package --vulnerable で High が検出されたため更新する。
- Microsoft.AspNetCore.Components.WebAssembly.Server 10.0.10(GHSA-g27p-qf6j-c96c・GHSA-qjpx-jqm5-h2c4)
  ほか Microsoft.* の 10.0.10 をすべて 10.0.13 に揃える
- System.Security.Cryptography.Pkcs 9.0.4(SqlClient 6.1.1 経由。GHSA-846f-f764-76mx ほか)は
  Infrastructure から 10.0.13 を直接参照して上書きする
- AspNetCore.OpenApi 10.0.13 が要求する Microsoft.OpenApi 2.12.2 へ直接参照も上げる(NU1605 の解消)
- dotnet-ef ツールを EF Core と同じ 10.0.13 に揃える

Co-Authored-By: Claude Opus 5.5 <[email protected]>
単票API(POST api/maintenance-orders/{id}/record)と実績CSV maintenance-records が
全ロールに開いており、物流・品質の担当者でも在庫の引落し・保全指示の完了・
設備の稼働可能への復帰・治工具の寿命リセットができた。
Spec.md 7.4 の「異常・使用実績の記録は絞らない」は保全依頼(起票)までで、実績は含まない。

- API・CSVとも MesRoleGroups.MaintenanceManage で絞る
- 画面の実績登録フォームも同じ定数で隠す(取消ボタンと同じ範囲にまとめた)
- 作業者が単票・CSVのどちらでも登録できず、指示が完了しないことをテストで確認
- Spec.md 改訂144

Co-Authored-By: Claude Opus 5.5 <[email protected]>
- Dependabot(nuget・github-actions):毎週月曜 09:00(JST)に確認し、dev 向けの PR を作る。
  担当者を自分に割り当て、GitHub の通知(Inbox)と Assigned の一覧に出す。
  Microsoft.* / System.* は版ずれ(NU1605)を避けるため1つの PR にまとめる
- Vulnerability ワークフロー:push・PR と週1回の定期実行で、推移的な依存も含めて
  既知の脆弱性を一覧にし、見つかったら失敗させる。復元時の NuGet 監査は警告がエラー扱いで
  一覧が出る前に落ちるため、復元では止めてこの手順で判定する

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@msmsrep

msmsrep commented Oct 23, 2026

Copy link
Copy Markdown
Owner Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant