Issue #716 has been updated by Raphaël Ouazana.

The first part of your patch seems fine. Can you elaborate a little more on the 
value.getString()/value.getValue() part? I don't see why you need this change.
----------------------------------------
Bug #716: Moving executable-plugin stream reading into thread.
http://tools.lsc-project.org/issues/716

Author: Nicolas Desseauve
Status: Assigned
Priority: Normal
Assigned to: Raphaël Ouazana
Category: Plugin
Target version: Executable plugin - 1.1
Problem in version: 


Hello,

In the executable plugin, the streams from the executed script are read after 
the script return. This can lead to the script to stop due to full buffer and 
the process never end(see 
http://docs.oracle.com/javase/6/docs/api/java/lang/Process.html)

I've made a small patch which create two tread for reading the output and error 
streams of the script.

I've also changer the entryToBean method to use value.getString() instead of 
value.getValue() because it was incorrectly storing base64 encoded string in 
the entry, but maybe there is another problem elsewhere.




-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://tools.lsc-project.org/my/account
_______________________________________________________________
Ldap Synchronization Connector (LSC) - http://lsc-project.org

lsc-dev mailing list
[email protected]
http://lists.lsc-project.org/listinfo/lsc-dev

Reply via email to