upstream/icinga-powershell-framework
1
0
Fork 0
mirror of https://github.com/Icinga/icinga-powershell-framework.git synced 2026-02-03 04:09:29 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #683 from Icinga:fix/jea_username_length_check_exclude_domain

Browse source 
Fix: JEA installer to exclude domain from user name length check

The domain of the provided user for JEA installation should not be included in the length check.

Windows has a maximum of 20 digits per username, which can easily be exceeded if the domain is wrongly included.
This commit is contained in:
Lord Hepipud 2024-02-20 14:16:20 +01:00 committed by GitHub
commit 173161cd54
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 16 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
doc/100-General/10-Changelog.md
View file

@ -11,6 +11,10 @@ Released closed milestones can be found on [GitHub](https://github.com/Icinga/ic
[Issues and PRs](https://github.com/Icinga/icinga-powershell-framework/milestone/28) [Issues and PRs](https://github.com/Icinga/icinga-powershell-framework/milestone/28)
### Bugfixes
* [#683](https://github.com/Icinga/icinga-powershell-framework/pull/683) Fixes JEA installer to exclude domain from user name length check, which can easily exceed the Windows 20 digits username limit
### Enhancements ### Enhancements
* [#679](https://github.com/Icinga/icinga-powershell-framework/pull/679) Adds a new data provider for fetching process information of Windows systems, while sorting all objects based on a process name and their process id * [#679](https://github.com/Icinga/icinga-powershell-framework/pull/679) Adds a new data provider for fetching process information of Windows systems, while sorting all objects based on a process name and their process id

6
lib/core/jea/Install-IcingaJeaProfile.psm1
View file

@ -13,9 +13,11 @@ function Install-IcingaJEAProfile()
return; return;
} }
$IcingaUserInfo = Split-IcingaUserDomain -User $IcingaUser;
# Max length for the user name # Max length for the user name
if ($IcingaUser.Length -gt 20) { if ($IcingaUserInfo.User.Length -gt 20) {
Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUser; Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUserInfo.User;
return; return;
} }

6
lib/core/windows/Install-IcingaSecurity.psm1
View file

@ -17,9 +17,11 @@ function Install-IcingaSecurity()
return; return;
} }
$IcingaUserInfo = Split-IcingaUserDomain -User $IcingaUser;
# Max length for the user name # Max length for the user name
if ($IcingaUser.Length -gt 20) { if ($IcingaUserInfo.User.Length -gt 20) {
Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUser; Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUserInfo.User;
return; return;
} }

6
lib/core/windows/New-IcingaWindowsUser.psm1
View file

@ -13,9 +13,11 @@ function New-IcingaWindowsUser()
}; };
} }
$IcingaUserInfo = Split-IcingaUserDomain -User $IcingaUser;
# Max length for the user name # Max length for the user name
if ($IcingaUser.Length -gt 20) { if ($IcingaUserInfo.User.Length -gt 20) {
Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUser; Write-IcingaConsoleError 'The specified user name "{0}" is too long. The maximum character limit is 20 digits.' -Objects $IcingaUserInfo.User;
return @{ return @{
'User' = $null; 'User' = $null;