Repository navigation
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Certificate search limitations and relative-path lookup precedence need clarification to prevent failed lookups or unintended certificate selection.
2 open issues
What changed in this PR
Add .NET CLI reference documentation for NuGet HTTPS client certificate management, with links from existing command navigation.
Changes:
- Document the add, update, list, and remove commands with options, limitations, password guidance, and examples.
- Add navigation links and alphabetize the NuGet command lists.
- Link package-source setup to client certificate configuration.
| File | Description |
|---|---|
docs/navigate/tools-diagnostics/toc.yml |
Add certificate commands and alphabetize navigation. |
docs/core/tools/dotnet.md |
Add certificate commands to the CLI command table. |
docs/core/tools/dotnet-nuget-update-client-cert.md |
Document certificate configuration updates. |
docs/core/tools/dotnet-nuget-remove-client-cert.md |
Document configuration removal and what remains unchanged. |
docs/core/tools/dotnet-nuget-list-client-cert.md |
Document certificate listing and output. |
docs/core/tools/dotnet-nuget-add-source.md |
Link to client certificate setup. |
docs/core/tools/dotnet-nuget-add-client-cert.md |
Document file-based and store-based certificate setup. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add reference pages for
dotnet nuget add,update,list, andremove client-cert. Document file and certificate-store options, password handling, command limitations, and examples. Add navigation links and alphabetize the NuGet command lists.Related to NuGet/Home#13485. This PR covers the .NET CLI documentation, not the NuGet CLI or standalone conceptual documentation.
Sources and AI usage: Adapt the four command sections from the accepted specification, checked against NuGet/NuGet.Client#3273 and the current implementation. No content is copied verbatim. AI-generated explanations and examples are marked with
ai-usage: ai-generated. The file-certificate--configfilerequirement warrants maintainer review because it reflects an implementation limitation.Validation: Executed all command examples on Windows with .NET SDKs 8.0.426, 9.0.319, and 10.0.402:
Also verified duplicate and mixed-option errors, certificate-type restrictions,
FindByprefix rejection,--forcebehavior, and local links and navigation. Historical SDKs, Linux, macOS, and live HTTPS authentication were not tested.Internal previews
Build report