You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mina.apache.org by "Guillaume Nodet (JIRA)" <ji...@apache.org> on 2013/07/19 09:46:51 UTC

[jira] [Resolved] (SSHD-224) Provide an inverted input stream for client channels

     [ https://issues.apache.org/jira/browse/SSHD-224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Guillaume Nodet resolved SSHD-224.
----------------------------------

    Resolution: Fixed

I've modified the ClientChannel to provide an inverted (OutputStream) input channel so that clients can directly write to the channel without having the need to start a thread and pump the stream.
                
> Provide an inverted input stream for client channels
> ----------------------------------------------------
>
>                 Key: SSHD-224
>                 URL: https://issues.apache.org/jira/browse/SSHD-224
>             Project: MINA SSHD
>          Issue Type: Bug
>         Environment: OSGi
>            Reporter: Mark Peters
>            Assignee: Guillaume Nodet
>             Fix For: 0.9.0
>
>
> Each ChannelSession creates its own thread to pump the data streams.  Currently, the only action it takes to stop this thread on doClose() is to interrupt the thread.  This isn't sufficient, and this is well known as evidenced by this comment:
>         // Interrupt does not really work and the thread will only exit when
>         // the call to read() will return.  So ensure this thread is a daemon
>         // to avoid blocking the whole app
>         streamPumper.setDaemon(true);
> Unfortunately, that's only a solution in simple applications.  When you have a plug-in architecture like OSGi where bundles come and go, this causes severe leaks as the threads are never cleaned up, even after you've closed the sessions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira