Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to pipe InputStream to ProcessBuilder

Please move down to the 2nd update. I didn't want to change the previous context of this question.

I'm using wkhtmltoimage from a Java app.

The standard way of using it is - path-to-exe http://url.com/ image.png.

According to their docs, if we write a - instead of an input URL, the input shifts to STDIN.

I'm starting the process using ProcessBuilder -

ProcessBuilder pb = new ProcessBuilder(exe_path, " - ", image_save_path);

Process process = pb.start();

Now I'm unable to figure out how to pipe an input stream to this process.

I have a template file read into a DataInputStream, and I'm appending a string at the end:

DataInputStream dis = new DataInputStream (new FileInputStream (currentDirectory+"\\bin\\template.txt"));
byte[] datainBytes = new byte[dis.available()];
 dis.readFully(datainBytes);
 dis.close();

 String content = new String(datainBytes, 0, datainBytes.length);

 content+=" <body><div id='chartContainer'><small>Loading chart...</small></div></body></html>";

How do I pipe content to the STDIN of the process?

UPDATE---

Following the answer by Andrzej Doyle:

I've used the getOutputStream() of the process:

ProcessBuilder pb = new ProcessBuilder(full_path, " - ", image_save_path);

    pb.redirectErrorStream(true); 

    Process process = pb.start();         

    System.out.println("reading");

    BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(process.getOutputStream()));

    bw.write(content);

Doing so gives an error saying:

Exception in thread "main" java.io.IOException: The pipe has been ended

2nd UPDATE--------

The current code block is as such:

    try {
        ProcessBuilder pb = new ProcessBuilder(full_path, "--crop-w", width, "--crop-h", height, " - ", image_save_path);
        System.out.print(full_path+ "--crop-w"+ width+ "--crop-h"+ height+" "+ currentDirectory+"temp.html "+ image_save_path + " ");
        pb.redirectErrorStream(true); 

        Process process = pb.start(); 
        process.waitFor();
        OutputStream stdin = process.getOutputStream();

        BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(stdin));
// content is the string that I want to write to the process.

        writer.write(content);
        writer.newLine();  
        writer.flush();
        writer.close();


    } catch (Exception e) {
        System.out.println("Exception: " + e);
        e.printStackTrace();
    }

Running the above code gives me an IOException: The pipe is being closed.

What else do I need to do to keep the pipe open?

like image 426
Hrishikesh Choudhari Avatar asked Jun 28 '12 09:06

Hrishikesh Choudhari


4 Answers

Exception in thread "main" java.io.IOException: The pipe has been ended

This means the process you have started has died. I suggest you read the output to see why. e.g. did it give you an error.

like image 162
Peter Lawrey Avatar answered Oct 19 '22 14:10

Peter Lawrey


Is there a reason you are using DataInputStream to read a simple text file? From the Java documentation

A data input stream lets an application read primitive Java data types from an underlying input stream in a machine-independent way

It's possible that the way you are reading the file causes an EOF to be sent to the outputstream causing the pipe to end before it gets to your string.

You requirements seems to be to read the file simply to append to it before passing it on to the wkhtmltoimage process.

You're also missing a statement to close the outputstream to the process. This will cause the process to wait (hang) until it gets an EOF from the input stream, which would be never.

I'd recommend using a BufferedReader instead, and writing it directly to the outputstream before appending your additional string. Then call close() to close the stream.

ProcessBuilder pb = new ProcessBuilder(full_path, " - ", image_save_path);
pb.redirectErrorStream(true);

Process process = null;
try {
    process = pb.start();
} catch (IOException e) {
    System.out.println("Couldn't start the process.");
    e.printStackTrace();
}

System.out.println("reading");

try {
    if (process != null) {
        BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(process.getOutputStream()));

        BufferedReader inputFile = new BufferedReader(new InputStreamReader(new FileInputStream(currentDirectory+"\\bin\\template.txt")));

        String currInputLine = null;
        while((currInputLine = inputFile.readLine()) != null) {
            bw.write(currInputLine);
            bw.newLine();
        }
        bw.write("<body><div id='chartContainer'><small>Loading chart...</small></div></body></html>");
        bw.newLine();
        bw.close();
    }
} catch (IOException e) {
    System.out.println("Either couldn't read from the template file or couldn't write to the OutputStream.");
    e.printStackTrace();
}

BufferedReader br = new BufferedReader(new InputStreamReader(process.getInputStream()));

String currLine = null;
try {
    while((currLine = br.readLine()) != null) {
        System.out.println(currLine);
    }
} catch (IOException e) {
    System.out.println("Couldn't read the output.");
    e.printStackTrace();
}
like image 34
Rajesh J Advani Avatar answered Oct 19 '22 14:10

Rajesh J Advani


Remove the whitespace from " - " -- the normal whitespaces are removed by the shell parser, but here in the ProcessBuilder, it's interpreted as the (literal) filename beginning and ending with a whitespace.

(Actually looking at the process's output as Peter suggested would probably have told you so...)

like image 38
Volker Stolz Avatar answered Oct 19 '22 14:10

Volker Stolz


After you create the Process object, you can call getOutputStream() in order to get hold of a stream that sends its contents to the process' standard input.

Once you have hold of this you can use Java's standard IO to write whatever bytes to the this stream you want (including wrapping it in a Writer, adding buffering, etc.) - and as you write them, they'll be read by the process as soon as they're flushed.

like image 37
Andrzej Doyle Avatar answered Oct 19 '22 16:10

Andrzej Doyle