CycloneDX / CycloneDX/cyclonedx-core-java

Dependencies within Dependency class should be List<String> not List<Dependency>

オープン
#199 コメント 4 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
120
フォーク
90
平均マージ
12時間 43分
マージ済み PR(30日)
18

説明

the `Dependency` class currently contains a dependencies field which is typed as a `List`, which implies that dependencies can be nested. This is incorrect based on both the current specification and the JSON schema, which has "dependsOn" as a list of bom-refs. This can lead to developers believing dependencies can be nested, and producing invalid BOMs as a result.

To double check this, I also compared this with the JavaScript which does implement the dependencies list as a string.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず Dependency クラスを見つけ、dependsOn が bom-refs のリストであることを踏まえて、その dependencies フィールドを現在の仕様および JSON スキーマと照合します。対応する JavaScript 実装と比較し、その後、Java モデルが依存関係参照を文字列として表現し、ネストされた依存関係をもはや示唆していないことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
backend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。