Skip to content

Commit

Permalink
Fixes usage of wrong certutil if present in PATH instead of system32 …
Browse files Browse the repository at this point in the history
…binary
  • Loading branch information
LordHepipud committed Apr 24, 2024
1 parent f32f8a4 commit cb63d5c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion doc/100-General/10-Changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,14 @@ Released closed milestones can be found on [GitHub](https://github.com/Icinga/ic

[Issues and PRs](https://github.com/Icinga/icinga-powershell-framework/milestone/32)

## 1.12.3 (tbd)
## 1.12.3 (2024-04-24)

[Issues and PRs](https://github.com/Icinga/icinga-powershell-framework/milestone/35)

### Bugfixes

* [#718](https://github.com/Icinga/icinga-powershell-framework/issues/718) Fixes Icinga repository JSON validator to report the correct state of the validation status, in case the JSON is not valid
* [#724](https://github.com/Icinga/icinga-powershell-framework/issues/724) Fixes `icingaforwindows.pfx` creation in case another `certutil` is installed on the system and being available in the `PATH` environment
* [#725](https://github.com/Icinga/icinga-powershell-framework/pull/725) Fixes Icinga for Windows certificate handling by keeping the .pfx file on the system as created, without using the certificate store. Also fixes handling for providing thumbprints for certificates, which are now loaded directly from the certificate store by also providing a new filter mechanic to fetch the proper certificates from the store

## 1.12.2 (2024-04-10)
Expand Down
2 changes: 1 addition & 1 deletion lib/webserver/ConvertTo-IcingaX509Certificate.psm1
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ function ConvertTo-IcingaX509Certificate()
# it is a temp file or we force its creation
if (-Not (Test-Path $TargetFile) -Or $TempFile -Or $Force) {
Write-Output "$Password
$Password" | certutil -mergepfx "$CertFile" "$TargetFile" | Set-Variable -Name 'CertUtilOutput';
$Password" | & 'C:\Windows\system32\certutil.exe' -mergepfx "$CertFile" "$TargetFile" | Set-Variable -Name 'CertUtilOutput';
}

Write-IcingaDebugMessage -Message (
Expand Down

0 comments on commit cb63d5c

Please sign in to comment.