Skip to content
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

Quickstart Guide should not create a self-signed certificate if a CertificateThumbprint is passed #281

Open
2 tasks done
JPRuskin opened this issue Nov 8, 2024 · 0 comments · May be fixed by #283
Open
2 tasks done

Comments

@JPRuskin
Copy link
Member

JPRuskin commented Nov 8, 2024

Checklist

  • I have verified this is the correct repository for opening this issue.
  • I have verified no other issues exist related to my request.

Is Your Feature Request Related To A Problem? Please describe.

If running QSG, a self-signed certificate is generated during the install of Chocolatey-Management-Service regardless of if a CertificateThumbprint is passed in to the scripts.

Describe The Solution. Why is it needed?

Having an additional (unused) certificate on the system adds confusion and an additional point of failure.

Additional Context

Currently, the code is installing the package the same way but calling Set-CCMCertificate if there's a thumbprint passed. The package creates a certificate if no thumbprint is passed to it. We should pass the thumbprint in to the package.

Related Issues

No response

JPRuskin added a commit that referenced this issue Nov 25, 2024
The Chocolatey-Management-Service package generates a self-signed certificate to use if no thumbprint is specified as a parameter.

Regardless of if a certificate was specified, we were installing it without a thumbprint and then setting the certificate later.

This resulted in an unused self-signed certificate being generated and stored.

This change ensures the package gets the appropriate parameter in order to not generate unneeded certificate(s).
@JPRuskin JPRuskin linked a pull request Nov 25, 2024 that will close this issue
3 tasks
JPRuskin added a commit that referenced this issue Dec 13, 2024
The Chocolatey-Management-Service package generates a self-signed certificate to use if no thumbprint is specified as a parameter.

Regardless of if a certificate was specified, we were installing it without a thumbprint and then setting the certificate later.

This resulted in an unused self-signed certificate being generated and stored.

This change ensures the package gets the appropriate parameter in order to not generate unneeded certificate(s).
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 a pull request may close this issue.

1 participant