Add errorhandling;Fix OU
	
		
			
	
		
	
	
		
	
		
			All checks were successful
		
		
	
	
		
			
				
	
				continuous-integration/drone/push Build is passing
				
			
		
		
	
	
				
					
				
			
		
			All checks were successful
		
		
	
	continuous-integration/drone/push Build is passing
				
			This commit is contained in:
		@@ -1,7 +1,7 @@
 | 
			
		||||
Name: 'COMP: Restrict Internet Communication'
 | 
			
		||||
Type: Object
 | 
			
		||||
LinkedOUs:
 | 
			
		||||
- OU=Servers
 | 
			
		||||
- OU=Servers,OU=Computer accounts
 | 
			
		||||
WMIFilters: []
 | 
			
		||||
RegistryEntries:
 | 
			
		||||
- Key: HKLM\Software\Policies\Microsoft\InternetManagement
 | 
			
		||||
 
 | 
			
		||||
@@ -175,16 +175,22 @@ If (@('primary','standalone') -contains $Parameter['deployment.type']) {
 | 
			
		||||
 | 
			
		||||
            ForEach ($OU in $GroupPolicy.LinkedOUs) {
 | 
			
		||||
                If (Test-Path "AD:\$($OU + (',{0}' -f (Get-ADRootDSE).rootDomainNamingContext))") {
 | 
			
		||||
                    Write-Host "Linking policy '$($NewGPO.DisplayName)' to OU '$($OU)' ..."
 | 
			
		||||
                    $NewGPLinkSplat = @{
 | 
			
		||||
                        Name        = $NewGPO.DisplayName
 | 
			
		||||
                        Target      = $OU + (',{0}' -f (Get-ADRootDSE).rootDomainNamingContext)
 | 
			
		||||
#                        ErrorAction = 'SilentlyContinue'
 | 
			
		||||
                    Try {
 | 
			
		||||
                        Write-Host "Linking policy '$($NewGPO.DisplayName)' to OU '$($OU)' ..."
 | 
			
		||||
                        $NewGPLinkSplat = @{
 | 
			
		||||
                            Name        = $NewGPO.DisplayName
 | 
			
		||||
                            Target      = $OU + (',{0}' -f (Get-ADRootDSE).rootDomainNamingContext)
 | 
			
		||||
                        }
 | 
			
		||||
                        New-GPLink @NewGPLinkSplat | Out-Null
 | 
			
		||||
                    }
 | 
			
		||||
                    Catch {
 | 
			
		||||
                        $ParseErrors += "Could not link GPO '$($NewGPO.DisplayName)' to OU '$($OU)'"
 | 
			
		||||
                        Continue
 | 
			
		||||
                    }
 | 
			
		||||
                    New-GPLink @NewGPLinkSplat | Out-Null
 | 
			
		||||
                }
 | 
			
		||||
                Else {
 | 
			
		||||
                    Throw "Path not accessible: 'AD:\$($OU + (',{0}' -f (Get-ADRootDSE).rootDomainNamingContext))"
 | 
			
		||||
                    $ParseErrors += "Path not accessible (referred to by '$($NewGPO.DisplayName)'): 'AD:\$($OU + (',{0}' -f (Get-ADRootDSE).rootDomainNamingContext))'"
 | 
			
		||||
                    Continue
 | 
			
		||||
                }
 | 
			
		||||
            }
 | 
			
		||||
        }
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user