Не срабатывает скрипт доступности доменных пк. Почему?

Question

Digsecman @Digsecman

PowerShell

Не срабатывает скрипт доступности доменных пк. Почему?

Пытаюсь получить список всех пк с сохранением в csv добавив поле-флаг "установлено", скрипт должен перебирать строчки в файле в поисках "не установлено" и "доступно" и если комп онлайн тогда инсталить если нет тогда пинговать и как только он будет в сети тогда установить по.
кроме того проверяется есть ли папка fusion, если нет тогда копируется содержимое с сетевой шары домена
помогите пожалуйста кто разбирается синтаксис поправить чтобы отрабатывало.

$dest="\\$all\C$\fusion"
$sourcefile = "\\WIN\c$\fusion\*"
$all=Get-ADComputer -LDAPFilter "(&(objectCategory=computer)(!userAccountControl:1.2.840.113556.1.4.803:=8192))" -Properties name | Sort).name|Import-csv C:\Users\user\Desktop\list.csv|
foreach ($comp in $all) {
  if (-not $comp.installed -and(test-connection $comp.name) -and(-not (test-path "C:\fusion\*"))) { 
if (!(Test-Path -path $dest))
    {
        New-Item $dest -Type Directory
    }
    Copy-Item -Path $sourcefile -Destination $dest;
     {Invoke-command -computername $comp.name -ScriptBlock –ThrottleLimit 120 {powershell "& 'C:\fusion\fusion.bat'"} 
     }
     $comp.installed=$true
  }
}
$all | export-csv C:\Users\user\Desktop\installed.csv

Вопрос задан более трёх лет назад
93 просмотра

4 комментария

Подписаться 1 Сложный 4 комментария

Alexey Dmitriev @SignFinder

Вам что мешает построчно его по F8 в ISE запускать, чтобы найти что именно не срабатывает?

Написано более трёх лет назад
Digsecman @Digsecman Автор вопроса

Alexey Dmitriev, Start-Transcript запускал но вписывая пропущенный символ в строчку все равно не отрабатывает, пробовал и так и сяк.
вот к примеру вывод:
At line:4 char:137
+ ... trol:1.2.840.113556.1.4.803:=8192))" -Properties name | Sort).name|Im ...
+ ~
Unexpected token ')' in expression or statement.
At line:5 char:16
+ foreach ($comp in $all) {
+ ~~
Unexpected token 'in' in expression or statement.
At line:5 char:15
+ foreach ($comp in $all) {
+ ~
Missing closing ')' in expression.
At line:5 char:23
+ foreach ($comp in $all) {
+ ~
Unexpected token ')' in expression or statement.
+ CategoryInfo : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : UnexpectedToken
выходит в строчке где foreach пропущены скобки но когда ставлю еще одну скобку после $all)) фейлиться по-прежнему

Написано более трёх лет назад
MaxKozlov @MaxKozlov Куратор тега PowerShell

когда я писал свой пример я сознательно каждое условие писал на своей строке. Зачем опять слеплять все в кучу?

вывод компов в файл и чтение из него были разными скриптами
И именно в этой строке дофига лишних скобок. Опять же, являющихся отсебятиной

Вы почему-то упорно не хотите следовать инструкциям хотя все время их просите.
.

Написано более трёх лет назад
Alexey Dmitriev @SignFinder

Digsecman, ответьте на мой вопрос. В какой строке у вас ошибка? Запустите его построчно, прверя переменные и т.п.

Написано более трёх лет назад

Вам что мешает построчно его по F8 в ISE запускать, чтобы найти что именно не срабатывает?
Alexey Dmitriev, Start-Transcript запускал но вписывая пропущенный символ в строчку все равно не отрабатывает, пробовал и так и сяк.
вот к примеру вывод:
At line:4 char:137
+ ... trol:1.2.840.113556.1.4.803:=8192))" -Properties name | Sort).name|Im ...
+ ~
Unexpected token ')' in expression or statement.
At line:5 char:16
+ foreach ($comp in $all) {
+ ~~
Unexpected token 'in' in expression or statement.
At line:5 char:15
+ foreach ($comp in $all) {
+ ~
Missing closing ')' in expression.
At line:5 char:23
+ foreach ($comp in $all) {
+ ~
Unexpected token ')' in expression or statement.
+ CategoryInfo : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : UnexpectedToken
выходит в строчке где foreach пропущены скобки но когда ставлю еще одну скобку после $all)) фейлиться по-прежнему
когда я писал свой пример я сознательно каждое условие писал на своей строке. Зачем опять слеплять все в кучу?

вывод компов в файл и чтение из него были разными скриптами
И именно в этой строке дофига лишних скобок. Опять же, являющихся отсебятиной

Вы почему-то упорно не хотите следовать инструкциям хотя все время их просите.
.
Digsecman, ответьте на мой вопрос. В какой строке у вас ошибка? Запустите его построчно, прверя переменные и т.п.

Answer 1 · 2021-08-17 23:35:05

Привет.
Прошу правильно понять меня. У Вас плохо написан скрипт. Такая реализация имеет место быть, но чем проще и очевиднее пишете тем лучше. (лично мое мнение). Да и судя по всему вы сами запутались в нем же.

Переменная $all получает данные из AD по фильтру и еще что то импортирует. - Тут ок, Вам виднее что вы хотите найти.
Но вот ошибка в скобках 4 строчки вероятно в выделенном месте:
$all=Get-ADComputer -LDAPFilter "(&(objectCategory=computer)(!userAccountControl:1.2.840.113556.1.4.803:=8192))" -Properties (name | Sort).name|Import-csv C:\Users\user\Desktop\list.csv

Только обратите внимание, что Вы дальше делаете конвеер foreach. Зачем тут конвеер если данные находятся в переменной.
В самом foreach только if содержится.
Так можно делать, но не удивляйтесь что дальше крашится будет еще активнее. Вероятнее всего в ветке Test-Path -path $dest и далее. У вас нет ловли ошибок и работы с ними. Используйте try catch.

Как минимум Хост может не ответить Вам на Test-Path , а еще туча ошибок при копировании (а есть ли папка назначения на данном хосте куда вы батник копируете? Это тоже стоит проверить) или Invoke-command . MaxKozlov не даст соврать, на этих командлетах можно много говна поймать.

Ну и концовка "$all | export-csv C:\Users\user\Desktop\installed.csv" - дело ваше как вы будете отслеживать на каких машинах взлетел скрипт на каких нет, но рекомендую заморочится в логировании.

Итог: Перепишите скрипт с нуля. У Вас сложная фильтрация в которой вы сами видимо запутались. Разбейте ее на 2-3 шага в отдельные переменные. Лучше так чем часами сидеть и не понимать ничего.
Сделайте полноценные хотя бы if else, ведь ошибки неизбежны.

Не срабатывает скрипт доступности доменных пк. Почему?

Войдите на сайт