Как я могу оптимизировать мой PowerShell - LDAP Query? - PullRequest
5 голосов
/ 19 июня 2010

Эй, ребята, это для гуру PS. Я создал скрипт, который читает из CSV (или другого набора данных, но не публикует эту сторону) и создает пользователей в моей среде AD.

По сути, любой набор данных, переданный в сценарий, будет обработан, а затем будет создан пользователь, если он не существует. Если пользователь уже существует в AD, сценарий пропускает запись. Это скрипт CREATE only.

Это довольно медленно, и я хотел бы улучшить производительность при сохранении функциональности. Можете ли вы дать мне какие-либо советы о том, как я могу сделать это лучше?

import-csv "c:\PSScripts\LDAP\ADMigrate.csv" | ForEach-Object {

# Define the User OU 
$usersOU = [ADSI] "LDAP://ou=Students, dc=live,dc=tcicollege,dc=edu"

# Check for existing users
$existingUsers = ($usersOU.psbase.children | Where-Object {$_.psBase.schemaClassName -eq "User"} | Select-Object -expand Name)
$userQuery = $existingUsers -contains $_.'AccountName'
if ($userQuery) {
    echo $_.'AccountName' " already exists in Directory."
} else {

    # Create a new user
    $newUser = $usersOU.create("user","cn=" + $_.'AccountName')

    # Set Account AttributesAMAccountName 
    $newUser.Put("sAMAccountName", $_.'AccountName')
    $newUser.Put("givenName", $_.'FirstName')
    $newUser.Put("employeeID", $_.'StudentID')
    $newUser.Put("sn", $_.'LastName')
    $newUser.Put("department", $_.'Department')
    $newUser.Put("company", $_.'SyStudentID')
    $newUser.Put("UserPrincipalName", $_.'AccountName' + "@live.tcicollege.edu")
    $newUser.Put("mail", $_.'AccountName' + "@live.tcicollege.edu")
    $newUser.Put("displayName", $_.'LastName' + "," + " " + $_.'FirstName')

    # First Commit
    $newUser.SetInfo()
    $newUser.userAccountControl="66048"
    $newUser.Put("pwdLastset", -1)
    $newUser.SetPassword($_.'Password')

    # Final Commit
    $newUser.SetInfo()
    echo $_.'AccountName' " created successfully."
  }
}

Заранее благодарю за любую помощь, которую вы можете предложить.

1 Ответ

5 голосов
/ 19 июня 2010

Попробуйте статический метод Exists (), чтобы определить, существует ли пользователь в OU Students:

$user = [ADSI]::Exists("LDAP://cn=$($_.AccountName),ou=Students, dc=live,dc=tcicollege,dc=edu")
if(!$user)  
{      
   "create code goes here"  
}  

Значение $ usersOU является статическим, поэтому вы можете его убрать, поместите перед командой import-csv.

...