Hi Ashish.

I did so. It is now here

https://issues.apache.org/jira/browse/DIRMINA-984

This is a trivial change. I hope you guys could push it soon.

--
Dmitri Priimak


On Sat, Sep 6, 2014 at 6:04 AM, Ashish <[email protected]> wrote:

> Can you please raise a JIRA for this?
>
>
> On Fri, Sep 5, 2014 at 7:46 AM, Dmitri Priimak <[email protected]> wrote:
>
> > Hi All.
> >
> > Using Sftplet I encountered one small problem.
> > In current git repo in file
> >
> >
> > sshd-sftp/src/main/java/org/apache/sshd/sftp/subsystem/SftpSubsystem.java
> >
> > in lines 138 through 141 you have following code
> >
> >     public void setSession(ServerSession session) {
> >
> >         sftpLet.onConnect(this);
> >
> >         this.session = session;
> >
> >     }
> >
> >
> > This calls sftpLet.onConnect(...) before session is set, which means that
> > at that stage information such as user name of the
> >
> > person who logged in is not available in inside of onConnect(...) call
> even
> > though ServerServer already has this information.
> >
> > A simple swapping of just two lines should fix that, like so
> >
> >
> >     public void setSession(ServerSession session) {
> >
> >         this.session = session;
> >
> >         sftpLet.onConnect(this);
> >
> >     }
> >
> >
> > Can such change be made in your code or there is some reason why the
> > current code works this way?
> >
> >
> > --
> > Dmitri Priimak
> >
>
>
>
> --
> thanks
> ashish
>
> Blog: http://www.ashishpaliwal.com/blog
> My Photo Galleries: http://www.pbase.com/ashishpaliwal
>

Reply via email to