Paul_0801 0 Geschrieben 12. November 2020 Melden Teilen Geschrieben 12. November 2020 (bearbeitet) Hi zusammen, ich muss ich einer großen Struktur mit mehren OU Benutzer finden, welche in einer bestimmten Gruppe sind. Mein Power Shell Skript sieht bis jetzt so aus: $OU= Read-Host "Geben Sie die OU an " Write-Host "Die Ou $Ou wurde ausgewählt!" $StrukturOU= Get-ADOrganizationalUnit -SearchBase"OU=Name,DC=DomainDC=Net" -SearchScope OneLevel -LDAPFilter "(Name=$OU)" $alluser= Get-ADUser -SearchBase $StrukturOU.DistinguishedName -LDAPFilter "(&(sn=*)(givenname=*))" | Select Name foreach ($user in $alluser) { $true = Get-ADGroupMember $user where AdGroup 'Gruppennamen' } Das Ziel ist es eigentlich, das am Ende bei True die Leute stehen, welche in der Gruppe drinnen sind. Aber irgendwie funktioniert es nicht. Habt ihr Vorschläge? Viele Grüße Paul bearbeitet 12. November 2020 von Paul_0801 Zitieren Link zu diesem Kommentar
NilsK 2.957 Geschrieben 12. November 2020 Melden Teilen Geschrieben 12. November 2020 Moin, also, zunächst solltest du natürlich Tippfehler vermeiden. "Struktur" ist nicht dasselbe wie "Strucktur". Das Wort "Gruop" wird die PowerShell auch nicht kennen. Dann ist es auch keine gute Idee, eine Variable "true" zu nennen, das ist ein reseviertes Schlüsselwort. Das ist technisch an der Stelle zwar (leider) zulässig, aber es erschwert das Verständnis deines Codes. Und schließlich wäre noch mal zu prüfen, was du überhaupt willst. Dein Code, wenn er funktionierte, würde wohl alle User der OU durchgehen und dann versuchen, diejenigen, die in der Gruppe sind, in eine Variable zu schreiben. Ist es so gedacht? (Funktionieren wird es schon deshalb nicht, weil die Logik immer nur die Daten des letzten abgefragten Users in der Variable ablegen würde ...) Abgesehen davon, dass der Code noch einige Fehler hat: Warum machst du das so? Was soll mit den Werten hinterher geschehen? Was ist das übergeordnete Ziel? Gruß, Nils Zitieren Link zu diesem Kommentar
Dukel 455 Geschrieben 12. November 2020 Melden Teilen Geschrieben 12. November 2020 Das Get-ADOU und Get-ADUser kann man auch zusammenfassen. Zitieren Link zu diesem Kommentar
Paul_0801 0 Geschrieben 12. November 2020 Autor Melden Teilen Geschrieben 12. November 2020 vor 14 Minuten schrieb NilsK: Moin, also, zunächst solltest du natürlich Tippfehler vermeiden. "Struktur" ist nicht dasselbe wie "Strucktur". Das Wort "Gruop" wird die PowerShell auch nicht kennen. Dann ist es auch keine gute Idee, eine Variable "true" zu nennen, das ist ein reseviertes Schlüsselwort. Das ist technisch an der Stelle zwar (leider) zulässig, aber es erschwert das Verständnis deines Codes. Und schließlich wäre noch mal zu prüfen, was du überhaupt willst. Dein Code, wenn er funktionierte, würde wohl alle User der OU durchgehen und dann versuchen, diejenigen, die in der Gruppe sind, in eine Variable zu schreiben. Ist es so gedacht? (Funktionieren wird es schon deshalb nicht, weil die Logik immer nur die Daten des letzten abgefragten Users in der Variable ablegen würde ...) Abgesehen davon, dass der Code noch einige Fehler hat: Warum machst du das so? Was soll mit den Werten hinterher geschehen? Was ist das übergeordnete Ziel? Gruß, Nils Oh, also mal von den Tippfehlern von angesehen. Genau, am Ende sollen alle User einer OU die in einer Gruppe sind ausgeben werden. Ich wollte am Ende die Variable in eine csv Datei ausgeben. Zitieren Link zu diesem Kommentar
NilsK 2.957 Geschrieben 12. November 2020 Melden Teilen Geschrieben 12. November 2020 (bearbeitet) Moin, ja, und dann? Was soll damit geschehen? Was ist die Anforderung? Mir erscheint der ganze Ansatz ziemlich umständlich. Vielleicht gibt es aber einen Grund dafür. Sonst würde ich dir raten, das gar nicht per PowerShell zu machen, sondern z.B. mit adfind. Gruß, Nils PS. in einem Forum gehört ein bisschen mehr Sorgfalt beim Tippen zur Höflichkeit. Beim Scripting geht es gar nicht ohne. Und wenn du Fragen zu deinem Code stellst, dann bitte mit exaktem Code (und auch als Code formatiert) - sonst raten wir hier nur rum. Danke. bearbeitet 12. November 2020 von NilsK 1 Zitieren Link zu diesem Kommentar
Empfohlene Beiträge
Schreibe einen Kommentar
Du kannst jetzt antworten und Dich später registrieren. Falls Du bereits ein Mitglied bist, logge Dich jetzt ein.