Question Module SSH

Salut Fabien,
benduru écrit:

Si vous avez des remarques ou des optimisations à proposer je suis évidement preneur

Moi m'sieu, moi j'en ai...

Le nom de module SSH est suffisant, la décompresion de l'archive ne demande rien d'autre comme manip.

Lors d'une premier installe, la vérification des prérequis ne serait pas du luxe.

ipmo ssh -verbose
COMMENTAIRES : Chargement du module à partir du chemin
« C:\Users\Laurent\Documents\WindowsPowerShell\Modules\ssh\ssh.psm1 ».
Exception lors de l'appel de « LoadFile » avec « 1 » argument(s) : « Le fichier spécifié est introuvable. (Exception de
HRESULT : 0x80070002) »
Au niveau de C:\Users\Laurent\Documents\WindowsPowerShell\Modules\ssh\ssh.psm1 : 34 Caractère : 32
+ [Reflection.Assembly]::LoadFile <<<< ('C:\Temp\Renci.SshNet35.dll') | out-null
+ CategoryInfo : NotSpecified: ( [], MethodInvocationException
+ FullyQualifiedErrorId : DotNetMethodException

COMMENTAIRES : Importation de la fonction « Get-SSHFile ».
COMMENTAIRES : Importation de la fonction « Get-SSHList ».
COMMENTAIRES : Importation de la fonction « Invoke-SSHCommand ».
COMMENTAIRES : Importation de la fonction « New-SFTPSession ».
COMMENTAIRES : Importation de la fonction « New-SSHSession ».
AVERTISSEMENT : Certains noms de commandes importés contiennent des verbes non approuvés qui les rendent moins
détectables. Utilisez le paramètre Verbose pour plus d'informations ou tapez Get-Verb pour obtenir la liste des verbes
approuvés.
COMMENTAIRES : Le nom de commande « Put-SSHFile » contient un verbe non approuvé qui le rend moins détectable. Les
verbes de remplacement suggérés sont « Send, Write ».
COMMENTAIRES : Importation de la fonction « Put-SSHFile ».
COMMENTAIRES : Importation de la fonction « Remove-SFTPSession ».
COMMENTAIRES : Importation de la fonction « Remove-SSHSession ».

Deux liens sur les conventions de nommage de PS, les verbes et les paramètres .

Côté code, je ne comprend pas le test, dans les lignes suivantes de la fonction 'New-SSHSession' :
[code:1]
Try {
$Client.Connect()
if ($Client -and $Client.IsConnected) {
[/code:1]
Ensuite, qq valeurs de tests sur les paramètres des différentes fonctions te remonteraient qq erreurs.

Je viens de tester rapido , tu devrais refaire une passe sur les exemples donnés. Celui -ci ne fonctionne pas :
[code:1]
Get-SSHFile -Client $Client -File \"/var/www/text/truc\" -Destination \"C:\temp\machin.bidule\"
[/code:1]
Tu as ajouté des paramètre mais pas la doc associée.

Pour ceci, dans Get-SSHFile,:
[code:1] $FileStream.Flush()
$FileStream.Close()
$FileStream.Dispose()[/code:1] il est préférable d'utilser une bloc Finally. Et souvent en .NET la méthode Close() appel en interne Dispose, consulte la doc pour être sûr, ce qui fait que les 2 appels sont redondants.

Et pour le support, comme c'est toi qui le fait autant te faciliter la vie ('TON script y marche pas!'), la clause Catch ne contient pas la cause réelle de l'erreur.
Tu peux utiliser -ErroRecord un exemple +- lié :
[code:1]
$PSCmdlet.WriteError(
(New-Object System.Management.Automation.ErrorRecord(
(New-Exception $_.Exception ($TextMsgs.ReplaceRegExCreate -F $_.Exception.Message)),
\"ReplaceRegExCreate\",
\"InvalidOperation\",
(\"[{0}]\" -f $Key)
)
)
)#WriteError
[/code:1]
Cela demande beaucoup plus de code, mais le gain se fait lors de la recherche de la cause du bug.
Ou comment perdre du temps pour en gagner

Enfin tu devrais livrer ton module avec la dll dépendante et la charger en relatif avec $PSScriptRoot.
Une fois ceci fait tu livres aux admins une solution et pas une source de 'problèmes'