PyCQA/bandit

yaml_load should not be B5xx cryptography group

Open

#306 opened on 2018年5月14日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)Python (559 forks)batch import
buggood first issue

Repository metrics

Stars
 (5,660 stars)
PR merge metrics
 (平均マージ 11h 55m) (30d で 1 merged PR)

説明

Describe the bug The yaml_load plugin has bandit ID B506. The 5xx group according to [1] is defined as the group for cryptography. This plugin would be more appropriate as a type of injection B6xx

To Reproduce n/a

Expected behavior n/a

Bandit version

bandit 1.4.0

Additional context Add any other context about the problem here.

コントリビューターガイド

調査方針
現在の yaml load プラグインは bandit ID B506 を使用しており、これは暗号化グループ（B5xx）に属しています。問題によれば、これはインジェクショングループ（B6xx）に分類されるべきです。修正するには、bandit/plugins/yaml load.py 内のプラグイン定義を探し、'id' フィールドを 'B506' から適切な B6xx ID（例：B601）に変更し、関連するテストファイルを更新してください。重複を避けるため、既存の B6xx ID については bandit のドキュメントを確認してください。これ以上の議論や PR はなく、この変更は単純で独立しています。
技術スタック
python
領域
security
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
Python
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80