fix(docker): correct stale --db-path comments in fetch/git Dockerfiles and clarify git README

オープン 初心者向け
#5,483 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

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

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
88/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
docker, typescript

調査の方向性

引用された箇所にある src/fetch/Dockerfile、src/git/Dockerfile、src/git/README.md を読みます。fetch のデータベースパスに関するコメントを削除し、git の Dockerfile のコメントをホストのリポジトリディレクトリをマウントする説明に更新し、README のコンテナユーザーの説明を修正します。3つすべての説明が issue に記載されたセットアップと一致すれば完了です。

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

説明

bug v2
Which server?

More than one server

Server version

n/a

How did you run the server?

Docker image

Transport

Not applicable (a repository-level report)

Protocol era

Not applicable (a repository-level report)

MCP client and version

n/a

Operating system and runtime

Docker / Linux

Server configuration

No response

Steps to reproduce
  1. Inspect src/fetch/Dockerfile at line 38:
    # when running the container, add --db-path and a bind mount to the host's db file
  2. Inspect src/git/Dockerfile at line 41:
    # when running the container, add --db-path and a bind mount to the host's db file
  3. Inspect src/git/README.md at line 204:
    Mentions that "The image runs as root, so it goes at /root/.gitconfig:", whereas the Dockerfile sets up the non-root app user (useradd -rUM -s /usr/sbin/nologin app).
Expected behavior
  • src/fetch/Dockerfile: The --db-path comment should be removed as the fetch server is an HTTP client without database arguments or host DB mounts.
  • src/git/Dockerfile: The comment should refer to bind-mounting the host repository directory, not a database path.
  • src/git/README.md: Documentation should accurately reflect the container's user execution model.
Actual behavior

Both Dockerfiles retain a copy-pasted comment from database servers referencing --db-path and database mounting.

Logs or error output

Already prototyped a fix?

In src/fetch/Dockerfile:
Remove the outdated comment on line 38 (# when running the container, add --db-path and a bind mount to the host's db file).

In src/git/Dockerfile:
Update line 41 to reference mounting the host's git repository directory.

Before you submit
  • I searched existing issues and this is not a duplicate.
  • This is not a security vulnerability report (those go through the private advisory form).
主要言語
TypeScript
スター
91.7k
フォーク
11.9k
平均マージ
3時間 30分
マージ済み PR(30日)
204

環境構築

はじめの一歩

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

modelcontextprotocol/servers のほかの issue

modelcontextprotocol/servers の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

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

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