chef / chef/chef-server

PUT /policy_groups/GROUP/policies/NAME does not check if policy is the same

オープン
#653 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Status: To be prioritized Triage: Feature Request
主要言語
Erlang
スター
303
フォーク
211
平均マージ
1日 8時間
マージ済み PR(30日)
5

説明

If you do the following:

PUT /policy_groups/a/policies/p
{
  "name": "p",
  "revision_id": "1.0.0",
  "cookbook_locks": {},
  "run_list": []
}

And then do this:

PUT /policy_groups/b/policies/p
{
  "name": "p",
  "revision_id": "1.0.0",
  "cookbook_locks": {},
  "run_list": [ "mycookbook" ]
}

The policy endpoint silently ignores the input to the second PUT. It should 409 and let me know that the policy will not be exactly what I thought it would.

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まずエンドポイント PUT /policy_groups/GROUP/policies/NAME を追跡し、issue にある2つのリクエストを再現します。ポリシーの内容が競合する状態でポリシーグループを変更すると HTTP 409 が返され、ポリシーが要求された入力と一致できないことが説明されることを確認します。

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

評価

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

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

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