-
Notifications
You must be signed in to change notification settings - Fork 37
Modify certutil commands for certificate installation #38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Updated certutil commands to include the '-f' flag for adding certificates to the store. The flag was present in the previous edit, but at the wrong position causing an error. Added instructions for creating a batch file to install a specific code signing certificate.
|
@MarkusMichalski : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @erinstellato-ms@mbarickman |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Updates the offline SSMS certificate installation documentation to adjust certutil.exe usage and add an alternative batch-file-based installation path for the Microsoft Windows Code Signing PCA 2024 certificate.
Changes:
- Reorders
certutil.exearguments to place-fbefore-addstorefor adding root certificates from a layout folder. - Adds an alternative
certutil.exebatch-file example for installing the Microsoft Windows Code Signing PCA 2024 certificate into theCAstore.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```console | ||
| certutil.exe -f -addstore "CA" "[layout path]\certificates\Microsoft Windows Code Signing PCA 2024.crt)" | ||
| ``` |
Copilot
AI
Jan 30, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example certutil command has a syntax error and an inconsistent file location. It ends with an extra ')' (which will break the path), and it references [layout path]\certificates\... even though the preceding steps only say to copy the .crt to the offline machine (not into the layout's certificates folder). Update the command to use the correct filename/quoting and clarify (or adjust) where the .crt should be placed before running the batch file.
|
Thanks for reviewing the proposed changes. Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
Updated certutil commands to include the '-f' flag for adding certificates to the store. The flag was present in the previous edit, but at the wrong position causing an error. Added instructions for creating a batch file to install a specific code signing certificate.