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

Enable-SqlDscReportServerTls: New command proposal #2022

Open
johlju opened this issue May 5, 2024 · 1 comment
Open

Enable-SqlDscReportServerTls: New command proposal #2022

johlju opened this issue May 5, 2024 · 1 comment
Labels
command proposal The issue is proposing to add a new command to the module. help wanted The issue is up for grabs for anyone in the community.

Comments

@johlju
Copy link
Member

johlju commented May 5, 2024

Command proposal

A command to switch to having TLS turned on.

$invokeRsCimMethodParameters = @{
CimInstance = $reportingServicesData.Configuration
MethodName = 'SetSecureConnectionLevel'
Arguments = @{
Level = @(0, 1)[$UseSsl]
}
}
Invoke-RsCimMethod @invokeRsCimMethodParameters

Proposed parameters

Parameter Mandatory Data type Description Default value Allowed values
ParameterName Yes String Detailed description None None

Special considerations or limitations

Se ConfigurationSetting method - SetSecureConnectionLevel that this command should have an integer for backward compatibility, but probably could have a boolean ([Switch]) parameter since this probably will be used for newer versions only.

@johlju
Copy link
Member Author

johlju commented May 5, 2024

Propose that we instead make this two commands: Enable-SqlDscReportServerTls and Disable-SqlDscReportServerTls.

@johlju johlju changed the title Set-SqlDscReportServerSecureConnectionLevel: New command proposal Enable-SqlDscReportServerTls: New command proposal May 5, 2024
@johlju johlju added help wanted The issue is up for grabs for anyone in the community. command proposal The issue is proposing to add a new command to the module. labels May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
command proposal The issue is proposing to add a new command to the module. help wanted The issue is up for grabs for anyone in the community.
Projects
None yet
Development

No branches or pull requests

1 participant