更新用户 ProxyAddresses

Question

我尝试更新 AD 用户帐户的 ProxyAddresses 属性。我已经阅读了许多关于此的主题并应用了一种建议的方法 (this one)，但它对我不起作用。为什么？ 我使用了以下代码（它是更新更多用户数据的脚本的一部分）：

$ADUser = SearchADUser -Kogo "sAMAccountName -eq '$($WzorUser.sAMAccountName)'"
...
1.$ProxyOK = $false
2.$Proxies = $ADUser.ProxyAddresses
3.$Proxies | ForEach-Object {
4.  $_ = $_ -replace 'SMTP', 'smtp'
5.  if ($_ -match $NoweMail) {
6.      $_ = $_ -replace 'smtp', 'SMTP'
7.      $ProxyOK = $true
8.  }
9.}
10.if (!($ProxyOK)) { $Proxies += ("SMTP:$($NoweMail)") }
...
if (!([string]::IsNullOrEmpty($Proxies))) {
    $AttrToReplace.Add("ProxyAddresses", $Proxies)
Set-ADUser -Identity $ADUser.sAMAccountName -Server $ADDC @Attr #-PassThru -WhatIf

在 Proxies 上循环时，它的元素会被正确处理：所有字母都小写，如果新邮件已经存在，则字母大写。 但是代理没有改变。每个元素是否需要以某种方式保存或替换到对象中？

2020-04-02 更新

由于支持者的努力（@Theo 再次感谢您的协助）专注于替换方法，我尝试更详细地解释我的问题。

目标用户 proxyAddresses 值（女性回到 AD 中已记录的娘家姓的情况很少见）：
SMTP:anna.nowak22@lp.pl
smtp：a.b@moc.com

初始值（第 2 行）：

[DBG]: PS X:\>> $Proxies
smtp:anna.nowak22@lp.pl
SMTP:a.b@moc.com

在每个代理元素上循环时（第 9 行）：

[DBG]: PS X:\>> $_
SMTP:anna.nowak22@lp.pl
[DBG]: PS X:\>> $Proxies
smtp:anna.nowak22@lp.pl
SMTP:a.b@moc.com

[DBG]: PS X:\>> $_
smtp:a.b@moc.com
[DBG]: PS X:\>> $Proxies
smtp:anna.nowak22@lp.pl
SMTP:a.b@moc.com

可以看出，$Proxies 没有反映变化。

如果只有一个 ProxyAddresses 值不等于新邮件，则将新邮件作为 SMTP 添加到现有邮件中，该邮件也保留为 SMTP（两个主要 ProxyAddresses）。

我尝试创建一个新变量并分别为其分配每个值，但我不知道如何处理它。

$newProxies = $null
$Proxies | ForEach-Object {
    $_ = $_ -creplace 'SMTP', 'smtp'
    if ($_ -match $NoweMail) {
        $_ = $_ -creplace 'smtp', 'SMTP'
        $ProxyOK = $true
    }
      $newProxies.add($_)
}

上面会产生错误

您不能在空值表达式上调用方法

$newProxies += $_ 创建一个字符串 SMTP:anna.nowak22@lp.pl smtp:a.b@moc.com 作为单个 ProxyAddress 添加。

正如我所指出的，$Proxies 是一个特殊的 AD 对象，我不知道如何创建这种类型的对象以及如何向其中添加新元素。

Answer 1

正如评论，这里作为答案更具可读性。

在您的代码中，您将修改后的$Proxies 数组添加到名为$AttrToReplace 的哈希表（我认为）中。但是，在最后的Set-ADUser 命令中，您并没有使用它，而是使用变量@Attr splat。

要更新多值属性 ProxyAddresses，请将问题中三个点后面的代码更改为：

if (!([string]::IsNullOrEmpty($Proxies))) {
    $AttrToReplace.Add("ProxyAddresses", $Proxies)
Set-ADUser -Identity $ADUser.sAMAccountName -Server $ADDC @Attr #-PassThru -WhatIf

进入：

# no need to test if $Proxies is an empty array, because 
# it will at the very least have "SMTP:$($NoweMail)"
Set-ADUser -Identity $ADUser.SamAccountName -Clear ProxyAddresses
Set-ADUser -Identity $ADUser.SamAccountName -Add @{proxyAddresses = $Proxies | ForEach-Object { "$_" }}

# or do this on one line:
# Set-ADUser -Identity $ADUser.SamAccountName -Replace @{proxyAddresses = $Proxies | ForEach-Object { "$_" }}

---

更新

从您的 cmets 来看，您的代码确实为重复的代理地址留出了空间。

您可以获取此列表，将所有现有的 SMTP: 替换为 smtp: 并将新的主电子邮件地址添加到其中，如下所示：

# get the current ProxyAddress values for this user, change all values 
# that begin with uppercase 'SMTP:' to lowercase 'smtp:'.
# skip any proxy that is equal to "smtp:$NoweMail", add the $NoweMail 
# preceeded by 'SMTP:' to the list and sort or select unique
$Proxies = @($ADUser.ProxyAddresses -replace '^SMTP:', 'smtp:' |
           Where-Object { $_ -ne "smtp:$NoweMail" }) + "SMTP:$NoweMail" | Sort-Object -Unique


# set the new proxy addresses in the user attribute
# no need to test if $Proxies is an empty array, because 
# it will at the very least have "SMTP:$($NoweMail)"
Set-ADUser -Identity $ADUser.SamAccountName -Clear ProxyAddresses
Set-ADUser -Identity $ADUser.SamAccountName -Add @{proxyAddresses = [string[]]$Proxies}

# or do this on one line:
# Set-ADUser -Identity $ADUser.SamAccountName -Replace @{proxyAddresses = [string[]]$Proxies}

^{请注意，在添加或替换的哈希中，使用的是 LDAP 名称，因此proxyAddresses 使用小写的p}

Answer 2

@西奥。您的解决方案工作正常。谢谢你的帮助。 我保持我的方法（它适合我并且有效）并稍微修改了代码。

$ProxyOK = $false
$Proxies = $null
$Proxies = @()
$ADUser.ProxyAddresses | ForEach-Object {
  $_ = $_ -creplace 'SMTP', 'smtp'
  if ($_ -match $NoweMail) {
      $_ = $_ -creplace 'smtp', 'SMTP'
      $ProxyOK = $true
  }
  $Proxies += $_
}
if (!($ProxyOK)) { $Proxies += ("SMTP:$($NoweMail)") }
...
if ($Proxies) {
    $AttrToReplace.Add("ProxyAddresses", [string[]]$Proxies)
}
if ($AttrToReplace.Count -gt 0) {
    $Attr.Add("Replace", $AttrToReplace)
}
Set-ADUser -Identity $ADUser.sAMAccountName -Server $ADDC @Attr #-PassThru -WhatIf