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

Extend and rework ImportSQL.ps1 #2673

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

merlinschumacher
Copy link

This change extends the ImportSQL.ps1 script, so that it automatically grants read permissions to the new cert. If the SQL server has no read permission for the private key, it will fail to run.
It also restructures the script a bit for easier debugging and extends the help.

This change extends the ImportSQL.ps1 script, so that it automatically grants read permissions to the new cert. If the SQL server has no read permission for the private key, it will fail to run.
It also restructures the script a bit for easier debugging and extends the help.
@WouterTinus
Copy link
Member

Hi @merlinschumacher! Thanks for this contribution, it looks like a good upgrade for the script. But I won't be merging any more PRs in this repo, because I'm working on a new cross-platform version of the program called simple-acme instead. Would you please consider contributing this to https://github.com/simple-acme/simple-acme/?

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.

2 participants