Skip to content

Show defaults for --as-system-proxy and --install-cert in --help - #2089

Open
waldekmastykarz wants to merge 1 commit into
dotnet:mainfrom
waldekmastykarz:show-defaults-system-proxy-install-cert
Open

waldekmastykarz wants to merge 1 commit into
dotnet:mainfrom
waldekmastykarz:show-defaults-system-proxy-install-cert

Conversation

@waldekmastykarz

Copy link
Copy Markdown
Collaborator

devproxy --help made --as-system-proxy and --install-cert look like opt-in switches, but both default to true. This change shows the boolean argument and the default in the help text, and links to the system proxy docs:

--as-system-proxy <true|false>   Register Dev Proxy as the system proxy (default: true). See https://go.devproxy.net?go=system-proxy
--install-cert <true|false>      Install and trust the root certificate (default: true)

Only the help text changes. Parsing is the same, so bare --as-system-proxy still means true.

Resolves #2058

Closes dotnet#2058

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@waldekmastykarz
waldekmastykarz requested a review from a team as a code owner November 5, 2026 10:43
Copilot AI balanced review requested due to automatic review settings November 5, 2026 10:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues remain for the command-line help clarification.

No open issues 🎉

What changed in this PR

Clarifies Dev Proxy’s command-line help so --as-system-proxy and --install-cert no longer appear to be opt-in switches. Parsing behavior is unchanged.

Changes:

  • Shows true|false arguments and documents that both options default to true.
  • Clarifies the option descriptions and links to the system proxy documentation.
File Description
DevProxy/​Commands/​DevProxyCommand.cs Updates help descriptions and argument labels for both boolean options.

Review effort: Balanced

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show defaults for --as-system-proxy and --install-cert in devproxy --help

2 participants