Remove deprecated config fields (proxyUrl, sslKeyPemPath, sslCertPemPath) in 3.0
メンテナーはふだん 1 日以内に返信
@re-vlad がすでに取り組んでいます。
2026年6月29日 から。
評価
この issue はまだ評価されていません。
説明
Summary
Remove the deprecated top-level config fields proxyUrl, sslKeyPemPath, and sslCertPemPath from the schema, types, and config loader. These fields are already flagged deprecated: true in config.schema.json and have been superseded by upstreamProxy and tls.key / tls.cert respectively.
Why defer to 3.0
Removing these fields is a breaking change: after the upgrade, the app will fail to start unless users clean up their config files. That's not acceptable inside the 2.x line, so this work should land in the next major.
Context
- Discussed in #1514 (review thread with @jescalada).
- The removal was briefly included in #1514 as commit
c48d0bfbut reverted in 4234c16 to keep #1514 non-breaking.
Scope
- Drop
proxyUrl,sslKeyPemPath,sslCertPemPathfromconfig.schema.json. - Regenerate
src/config/generated/config.ts. - Remove the legacy fallback logic in
src/config/index.ts(mergeConfigurations,getProxyUrl,getTLSKeyPemPath,getTLSCertPemPath). - Simplify
FullGitProxyConfigto plainRequired<GitProxyConfig>(dropOptionalTopLevelConfigKey). - Update reference docs (
website/docs/configuration/reference.mdx). - Improve the config-load error message so users hitting a removed/unknown key get a clear "this field is no longer supported, use X" instead of the current misleading "expected boolean but got X" (per @jescalada's note in #1514).
Migration for users
| Deprecated field | Replacement |
|---|---|
proxyUrl |
upstreamProxy |
sslKeyPemPath |
tls.key |
sslCertPemPath |
tls.cert |
- 主要言語
- TypeScript
- スター
- 263
- フォーク
- 185
- 平均マージ
- 2日 15時間
- マージ済み PR（30日）
- 39
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
finos/git-proxy のほかの issue
-
documentation good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
finos/git-proxy#1878 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
scanDiff ignores commitConfig.diff.block rules from the --config file対応中かも @goldensyntax が 3 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
finos/git-proxy#1874 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
checkUserPushPermission is checking the wrong user's permission to push再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンbug tech-debt
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
finos/git-proxy#1400 · コメント 17 件 ·
メンテナーはふだん 1 日以内に返信
-
meeting
難易度 1/5 1時間未満 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
finos/git-proxy の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
hugomrdias/playwright-test#753 ·
メンテナーはふだん 1 日以内に返信
-
Review view: Daily/Weekly Review header buttons use the app-wide toolbar control style対応中かも @colherdesopa が今日担当しました。 オープン
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
dongdongbh/Mindwtr#1515 ·
メンテナーはふだん 1 日以内に返信
-
pending-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
kirodotdev/Kiro#12301 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100