Faire son propre InputStream pour Jsch

Bonjour à vous,
je souhaite que mon programme envoie des commandes via SSH à mon serveur.
Pour ça j'utilise la librairie Jsch avec l'exemple suivant que j'ai modifié à ma sauce pour que je puisse me connecter en rentrant juste mon mot de passe.
Jusque là tout va bien, je peux naviguer sur mon serveur dans la console puisque l'entrée et la sortie sont gérées:

Code:

---

1 2 3

channel.setInputStream(System.in); channel.setOutputStream(System.out);

---

Mais je voudrais mettre mon propre InputStream pour envoyer les commandes que je veux au shell.
Donc j'ai fait ma classe:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42

public class InputStreamSSH extends InputStream{   private volatile String input;   public InputStreamSSH(){ input = new String(); }   public int available() throws IOException{ return input.length(); }   @Override public int read() throws IOException {   while(this.available() == 0){ try { Thread.sleep(100); } catch (InterruptedException e) { // TODO Auto-generated catch block e.printStackTrace(); } } int currentByte = input.getBytes()[0]; input = input.substring(1, input.length());     return currentByte;   }   public void write(String str){ input += str; }   public boolean markSupported(){ return false; }     }

---

en suivant la doc.
Mais par exemple si j'essaye un "mkdir toast" rien ne se crée sur mon serveur (chemin bien vérifié).
J'ai essayé en ajoutant le retour ligne et fin de ligne pour simuler la touche entrée, mais ça change rien:

Code:

---

1 2 3 4

final char CR = 13; final char FL = 10; return cmd + CR + FL;

---

Ma classe InputStream est sûrement mal faîte..
Ou j'ai oublié quelque chose ailleurs ?
J'ai bien mit le

Code:

---

1 2	channel.setInputStream(myInputStream);

---

Et pour appeler write(String) de ma classe InputStream j'utilise un autre Thread.
J'ai testé chaque méthode voir ses retours et de ce côté c'est bon.

Merci d'avance de vos réponses, et si y'a une solution qui existe déjà et que j'ai loupé je veux bien la connaître, ou même si j'ai fait une erreur !
Cordialement.

Code:

---

input = new String();

---

On ne fais jamais un new String en java, ça ne sert qu'à créer un objet inutile. Met directement la String que tu veux dedans, si t'as besoin d'une chaine vide:input ="";

Code:

---

while(this.available() == 0){

---

Il n'y a rien dans ton code qui fais que magiquement available va passer à autre chose que 0, cette boucle va juste boucler ad vitam.

Code:

---

input.getBytes()

---

Comportement imprévisible. La méthode getByte() transforme la chaine en utilisant l'encodage par défaut de ta plateforme, ça peux vouloir être tout et n'importe quoi. On précise toujours l'encodage à utiliser explicitement.


Si tu veux balancer des données dans un InputStream, utilise simplement PipedInputstream:

Code:

---

1 2 3 4 5 6 7 8 9

PipedInputStream pis = new PipedInputStream(); PipedOutputStream pos = new PipedOutputStream(pis); PrintStream ps = new PrintStream(pos,true,"UTF-8");   channel.setOutputStream(System.out); // sortie du channel channel.setInputStream(pis); // entrée du channel channel.connect();   pos.println("touch /tmp/test123");

---

A noter que tu peux te simplifier encore plus la vie en utilisant la méthode getOutputStream du channel:

Code:

---

1 2 3 4 5

channel.setOutputStream(System.out); // sortie du channel PrintStream ps = new PrintStream(channel.getOutputStream(),true,"UTF-8"); // entrée du channel channel.connect(); pos.println("touch /tmp/test123");

---

Salut,


Je dirais que tu ne redéfinies pas les méthode read(byte(]), et que l'implémentation par défaut ne doit pas bien fonctionner car ton read() ne retourne jamais -1...

Sans même parler des objets temporaires créer un peu partout :?


Mais surtout pourquoi s'embêter : les Piped(Input/Output)Stream permettent de faire cela simplement :

Code:

---

1 2 3 4

PipedInputStream input = new PipedInputStream(); channel.setInputStream(input);   PipedOutputStream output = new PipedOutputStream(input);

---

:arrow: Tout ce qui est écrit dans "output" sera lu dans "input"...



En plus tu peux l'englober dans un PrintStream pour l'utiliser plus simplement :

Code:

---

1 2 3

PrintStream output = new PrintStream(PipedOutputStream(input), true);   output.println("mkdir toast");

---

a++

Je connaissais pas le PipedInputStream, merci pour vos commentaires ;)