Invalid JSON in `CMD` is rewritten to exec form

オープン 初心者向け
  • 何か問題がありますか？
#79 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

評価

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

調査の方向性

まずGoのソースコードでCMDの処理とJSON引数の解析を見つけ、次にCMDのフォーマットに関するテストを読んでください。CMD ["a",]のケースを再現し、引数が有効なJSON配列でない場合にフォーマットによって変更されないことを確認してください。回帰テストを追加し、プロジェクトのテストスイートで成功することを確認してください。

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

説明

CMD with an argument list that is not valid JSON gets wrapped into exec form. Docker treats such a line as shell form, so the rewrite changes the command the container runs.

Reproduce
FROM scratch
CMD ["a",]

dockerfmt Dockerfile
Actual
FROM scratch
CMD ["[\"a\",]"]

BuildKit's parser (v0.33.0) reads the original as shell form (/bin/sh -c '["a",]'). The output is exec form that runs a binary literally named ["a",].

Expected

Leave arguments that do not parse as a JSON array unchanged, as Docker does.

Reproduced with v0.5.4 and with main at a5868c2.

主要言語
Go
スター
626
フォーク
23
平均マージ
6日 23時間
マージ済み PR（30日）
3

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

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

reteps/dockerfmt のほかの issue

reteps/dockerfmt の issue をすべて見る

似ている issue

Go の issue をもっと見る

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

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