Skip to content

Commit 18f5ad8

Browse files
authored
Merge pull request #177 from Icinga:fix/allow_domain_accounts_for_wmi_permissions
Fix: Allow usage for domain accounts for wmi permissions without being locally known
2 parents 0618d30 + 17e320d commit 18f5ad8

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

Diff for: doc/31-Changelog.md

+1
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ Released closed milestones can be found on [GitHub](https://github.com/Icinga/ic
4545
* [#165](https://github.com/Icinga/icinga-powershell-framework/pull/165) Fixes fetching for Icinga Agent certificate for REST-Api daemon on upper/lower case hostname mismatch
4646
* [#166](https://github.com/Icinga/icinga-powershell-framework/pull/166) Fixes fetching of Icinga Agent MSI packages by correctly comparing versions to ensure we always use the latest version and fixes `release` usage for local/network drive sources
4747
* [#167](https://github.com/Icinga/icinga-powershell-framework/pull/167) Fixes error while writing EventLog entries with too large message size
48+
* [#177](https://github.com/Icinga/icinga-powershell-framework/pull/177) Fixes Wmi permissions to allow domain accounts while not being locally known on the system
4849

4950
## 1.2.0 (2020-08-28)
5051

Diff for: lib/wmi/Get-IcingaWmiSecurityData.psm1

-7
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,6 @@ function Get-IcingaWmiSecurityData()
3939
$UserSID = Get-IcingaUserSID -User $User;
4040
$WmiAcl = $WmiSecurityData.Descriptor;
4141

42-
$WmiAccount = Get-IcingaWindowsInformation -ClassName Win32_Account -Filter ([string]::Format("Domain='{0}' and Name='{1}'", $UserData.Domain, $UserData.User));
43-
44-
if ($null -eq $WmiAccount) {
45-
Write-IcingaConsoleError 'The specified user could not be found on the system: "{0}\{1}"' -Objects $UserData.Domain, $UserData.User;
46-
return $null;
47-
}
48-
4942
if ([string]::IsNullOrEmpty($UserSID)) {
5043
Write-IcingaConsoleError 'Unable to load the SID for user "{0}"' -Objects $User;
5144
return $null;

0 commit comments

Comments
 (0)