You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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 feat(config): require merged top-level defaults #1514).
Summary
Remove the deprecated top-level config fields
proxyUrl,sslKeyPemPath, andsslCertPemPathfrom the schema, types, and config loader. These fields are already flaggeddeprecated: trueinconfig.schema.jsonand have been superseded byupstreamProxyandtls.key/tls.certrespectively.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
c48d0bfbut reverted in 4234c16 to keep feat(config): require merged top-level defaults #1514 non-breaking.Scope
proxyUrl,sslKeyPemPath,sslCertPemPathfromconfig.schema.json.src/config/generated/config.ts.src/config/index.ts(mergeConfigurations,getProxyUrl,getTLSKeyPemPath,getTLSCertPemPath).FullGitProxyConfigto plainRequired<GitProxyConfig>(dropOptionalTopLevelConfigKey).website/docs/configuration/reference.mdx).Migration for users
proxyUrlupstreamProxysslKeyPemPathtls.keysslCertPemPathtls.cert