diff --git a/src/Certificate/New-CSCertificate.ps1 b/src/Certificate/New-CSCertificate.ps1 index 6e93ef0..7cb2ef1 100644 --- a/src/Certificate/New-CSCertificate.ps1 +++ b/src/Certificate/New-CSCertificate.ps1 @@ -26,6 +26,11 @@ function New-CSCertificate { #> [CmdletBinding(SupportsShouldProcess = $true)] + [Diagnostics.CodeAnalysis.SuppressMessageAttribute( + 'PSAvoidUsingInvokeExpression', + '', + Justification = 'needed for openssl wrapping' + )] [OutputType()] param ( [Parameter(Mandatory = $true, ValueFromPipeline = $true)] @@ -57,7 +62,10 @@ function New-CSCertificate { Write-Error @ErrorParams } } - elseif (($PSVersionTable.PSEdition -eq 'Desktop' -and $PSVersionTable.PSVersion.Major -lt 6) -or ($IsWindows -eq $true)) { + elseif ( + ($PSVersionTable.PSEdition -eq 'Desktop' -and $PSVersionTable.PSVersion.Major -lt 6) -or + ($IsWindows -eq $true) + ) { $openssl = Join-Path -Path $ModuleBase -ChildPath '/Vendor/libressl255/openssl.exe' } @@ -77,7 +85,9 @@ function New-CSCertificate { $Subj = $SubjPattern -f $SubjValues $PEMCertName = $CertName -replace '.pfx', '.crt' - $ExpPattern = '& ''{0}'' req -x509 -sha256 -nodes -days {1} -newkey rsa:2048 -keyout {2} -out {3} -subj "{4}" *>$null' + $ExpPattern = ( + '& ''{0}'' req -x509 -sha256 -nodes -days {1} -newkey rsa:2048 -keyout {2} -out {3} -subj "{4}" *>$null' + ) $ExpValues = @( $openssl, $CRTAttribute.Days diff --git a/src/Store/Get-CredentialStore.Tests.ps1 b/src/Store/Get-CredentialStore.Tests.ps1 index bb940a0..e6e1f5a 100644 --- a/src/Store/Get-CredentialStore.Tests.ps1 +++ b/src/Store/Get-CredentialStore.Tests.ps1 @@ -14,8 +14,6 @@ BeforeAll { Describe "Get-CredentialStore" { Context "Basic logic tests" { It "Read CS without params" { - $TestCredentialStore = './resources/cs/CredentialStore.json' - $TestPfxCert = './resources/cs/PSCredentialStore.pfx' if (! (Test-Path -Path (Get-DefaultCredentialStorePath)) ) { { New-CredentialStore -Force } | Should -Not -Throw } diff --git a/src/Store/New-CredentialStore.ps1 b/src/Store/New-CredentialStore.ps1 index d18433a..29056fb 100644 --- a/src/Store/New-CredentialStore.ps1 +++ b/src/Store/New-CredentialStore.ps1 @@ -19,13 +19,13 @@ function New-CredentialStore { Use this switch to reset an existing store. The complete content will be wiped. .PARAMETER SkipPFXCertCreation - You can skip the pfx certificate creation process. This makes sense if you have a previously created cert or want to - import a cert in cross-platform environments. + You can skip the pfx certificate creation process. This makes sense if you have a previously created cert + or want to import a cert in cross-platform environments. .Parameter UseCertStore - Instead of using a plain pfx file beside your CredentialStore file you can import it into the user or machine - certificate store. In this case the system itself secures the cert and you don't hat to set custom NTFS - permissions so secure your shared certificate. + Instead of using a plain pfx file beside your CredentialStore file you can import it into the user or + machine certificate store. In this case the system itself secures the cert and you don't hat to set custom + NTFS permissions so secure your shared certificate. .INPUTS [None] @@ -88,7 +88,8 @@ function New-CredentialStore { $ErrorParams = @{ ErrorAction = 'Stop' Exception = [System.IO.InvalidDataException]::new( - 'Please provide a full path containing the credential store file name with the .json extension!' + 'Please provide a full path containing the ' + + 'credential store file name with the .json extension!' ) } Write-Error @ErrorParams