Question A propos du script Get-Audit

Olivier, voici quelques remarques sur le script suivant : Get-Audit.ps1

Il me semble que tu utilises Write-Output au lieu de Write-Host, je me tompe ?
Tu envisageais qu'il \"soit accessible en plusieurs \"format\" qui pour l'instant est juste \"HTML\".
Dans ce cas, il faut, à mon avis, découpler l'action d'inventaire de sa présentation, qq chose comme:
[code:1]
Get-Audit|Format-AuditHtml|Out-Html
[/code:1]
Tu peux formater le nom de fichier avec l'opérateur -F :
[code:1]
$date=get-date
$Filename = $(\".\$Target {0:HH}h{0:mm} - {0:dd}.{0:MM}.{0:yyyy}.htm\" -f $Date)
$Filename
[/code:1]

La dernière ligne provoque une exception dans le cas suivant :
[code:1]
PS Env:\> &\"G:\temp\Get_Audit\Get-Audit.ps1\"
No list specified, using xxx
...
..Installed Software
Out-File : Impossible d'ouvrir le fichier, car le fournisseur actuel (Microsoft.PowerShell.Core\Environment) ne permet
pas cette opération.
Au niveau de G:\temp\Get_Audit\Get-Audit.ps1 : 821 Caractère : 19
+ $Report | out-file <<<< -encoding UNICODE -filepath $Filename
+ CategoryInfo : InvalidArgument: (:«») [Out-File], PSInvalidOperationException
+ FullyQualifiedErrorId : ReadWriteFileNotFileSystemProvider,Microsoft.PowerShell.Commands.OutFileCommand
[/code:1]
C'est du au fait que tu références le chemin courant $FileName; sous PS la notion de chemin est différente des shells habituels.
Un PsPath référence tout type de provider pas seulement le file system :
[code:1]
cd env:
$ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($pwd)
# chaine vide
cd c:
$ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($pwd)
#chemin courant sur c:
cd env:
$ExecutionContext.SessionState.Path.CurrentProviderLocation(\"filesystem\"«»)
#chemin courant du provider filesystem
[/code:1]
Et si c'est l'utilisateur qui précise le chemin, il y aurait d'autres contrôles à faire.

J'aurais plutot vu le type du paramètre comme un tableau de string, si par exemple les noms de pc proviennent d'AD cela impose de les écrire dans un fichier pour les réimporter.
Dans une version évoluée, tu pourrais aussi utiliser le pipeline :
[code:1]
Get-Machines|Get-audit
[/code:1]
Si les ordinateurs n'existe pas ou ne sont pas connectés le script génére pas mal d'erreurs WMI :
[code:1]
Get-WmiObject : Le serveur RPC n'est pas disponible.
[/code:1]
Il serait bien d'ajouter une gestion des erreurs WMI.

Pour les codes du type suivant, tu peux utiliser un switch ou des elseif :
[code:1]
If ($objService.StartMode -eq \"Auto\"«»)
[/code:1]

Tu concaténes beaucoup de chaînes autour de la variable $Report, l'usage d'un objet StringBuilder serait préférable.

Je constate des redondance dans les chaînes de type :
[code:1]
$Report+= \" <td width='20%'><font color='#FF0000'>$($objService.State)</font></td>\"
[/code:1]
Une approche comme celle-ci peut aider à éclaircir le code :
[code:1]
function New-tdHtmlBalise($width,$Color,[string]$ObjectProperty)
{\" <td width='{0}%'><font color='#{1}'>{2}</font></td>\" -F $width,$Color,$Object $ObjectProperty
New alias td New-tdHtmlBalise

$Report+= td 20 FF0000 $objService.State
#$Report.Add( (td 20 FF0000 $objService.State) )
[/code:1]

Et enfin pour ce type de code :
[code:1]
$Report+= \" <tr>\"
$Report+= \" <td width='25%'>$($objQuickFix.HotFixID)</font></td>\"
$Report+= \" <td width='75%'>$($objQuickFix.Description)</font></td>\"
$Report+= \" </tr>\"
[/code:1]
Tu peux utiliser une here-string :
[code:1]
$Report+=@\"
<tr>
<td width='25%'>$($objQuickFix.HotFixID)</font></td>
<td width='75%'>$($objQuickFix.Description)</font></td>
</tr>
\"@
[/code:1]
C'est tout

Salut,

Je n'ai pas encore eu le temps de regarder le code, j'ai juste testé le script...

Les suggestions d'amélioration que je propopose sont les suivantes :
1. Mettre la feuille de style dans un fichier CSS externe
2. Pouvoir exporter le résultat sous Word et/ou sous Excel
3. Effectivement comme le propose Laurent, faire un test si la machine distante est joignable (Test-Connection) avant de tenter l'execution du script.

Arnaud

Salut,
tu peux consulter ce post
powershell-scripting.com/index.php?optio...=749&catid=5#749

Sinon pour ce type de test
[code:1]({Test-Path D:\Windows\PowerShell\Scripts\Working.txt} -eq \"true\"«»)[/code:1]
Il n'est pas nécessaire d'utiliser un scriptblock, des parenthèses suffisent :
[code:1]
((Test-Path D:\Windows\PowerShell\Scripts\Working.txt))[/code:1]
Par défaut Test-Path renvoi $True si le test réussi.
On utilise -eq $true dans certains cas pour préciser/documenter le code, mais ici c'est redondant.