Author psykiatris
Recipients jeff.allen, mga792, psykiatris
Date 2018-04-11.04:19:26
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
In-reply-to <>
Final report and recommendation:

1. If shell-True, subprocess will execute /bin/bash. Code works if an 
executable shell script and arguments are provided.

2. it is better to pass arguments in as a list. Example: cmd=['ping', 
';, '-c 10']. If you do this, you do not need to specify an 
executable= setting.

3. p.communicate() closes the stdin, stdout, stderr streams, collects 
all input for output. Inserting 10 newlines (\n) has no effect. It is 
appropriate to leave it empty, or use None.

4. The only way text can be inserted into the stdin is if the external 
program has a function to do the following: 
sys.stdout.write( This can be done internally with 
Jython and Python code.

I've attached the updated scrpt. My previous patches were 
poorly thought out and will not work. My apologies.

Recommendation: Mark this issue as invalid as the code works with no 
errors with the minor changes that were made.

On 04/07/2018 09:23 AM, Patrick Palczewski wrote:
> Patrick Palczewski <> added the comment:
> Yes, I saw in the documentation after I posted. It was pretty late. :/
File name Uploaded psykiatris, 2018-04-11.04:19:26
unnamed psykiatris, 2018-04-11.04:19:24
Date User Action Args
2018-04-11 04:19:26psykiatrissetrecipients: + psykiatris, jeff.allen, mga792
2018-04-11 04:19:26psykiatrislinkissue2664 messages
2018-04-11 04:19:26psykiatriscreate