Re: Moving new nbxml in trunk
Niklas Gustavsson wrote: > On Tue, May 4, 2010 at 5:34 PM, Bernd Fondermann > wrote: >> BTW, that's common in real life XMPP streams: arbitrary whitespaces. >> Some clients send spaces as keep-alives. > > The problem struck me right after sending the last email (after which > I had to run of to go home). Now fixed and test updated :-) Probably the test coverage is not sufficient. Did you actually try to connect a real client to a real server? Still getting errors: 10:43:13,219 | DEBUG | org.apache.vysper.xml.sax.impl.XMLParser | Fatal error: null java.lang.NullPointerException at org.apache.vysper.xml.decoder.XMPPContentHandler.characters(XMPPContentHandler.java:79) at org.apache.vysper.xml.sax.impl.XMLParser.characters(XMLParser.java:306) at org.apache.vysper.xml.sax.impl.XMLParser.token(XMLParser.java:142) at org.apache.vysper.xml.sax.impl.XMLTokenizer.emit(XMLTokenizer.java:156) at org.apache.vysper.xml.sax.impl.XMLTokenizer.parse(XMLTokenizer.java:81) at org.apache.vysper.xml.sax.impl.XMLParser.parse(XMLParser.java:123) at org.apache.vysper.xml.sax.impl.DefaultNonBlockingXMLReader.parse(DefaultNonBlockingXMLReader.java:180) at org.apache.vysper.xml.decoder.XMPPDecoder.doDecode(XMPPDecoder.java:96) at org.apache.mina.filter.codec.CumulativeProtocolDecoder.decode(CumulativeProtocolDecoder.java:178) at org.apache.mina.filter.codec.ProtocolCodecFilter.messageReceived(ProtocolCodecFilter.java:225) at org.apache.mina.core.filterchain.DefaultIoFilterChain.callNextMessageReceived(DefaultIoFilterChain.java:434) at org.apache.mina.core.filterchain.DefaultIoFilterChain.access$1200(DefaultIoFilterChain.java:46) at org.apache.mina.core.filterchain.DefaultIoFilterChain$EntryImpl$1.messageReceived(DefaultIoFilterChain.java:793) at org.apache.mina.core.filterchain.IoFilterEvent.fire(IoFilterEvent.java:71) at org.apache.mina.core.session.IoEvent.run(IoEvent.java:63) at org.apache.mina.filter.executor.OrderedThreadPoolExecutor$Worker.runTask(OrderedThreadPoolExecutor.java:780) at org.apache.mina.filter.executor.OrderedThreadPoolExecutor$Worker.runTasks(OrderedThreadPoolExecutor.java:772) at org.apache.mina.filter.executor.OrderedThreadPoolExecutor$Worker.run(OrderedThreadPoolExecutor.java:714) at java.lang.Thread.run(Thread.java:637) 1
[CANCEL] [VOTE] Release SSHD 0.4.0
Cancelling this vote. I've fixed the SFTP issue and will upload a new candidate asap. On Tue, May 4, 2010 at 15:19, Guillaume Nodet wrote: > I've uploaded a RC for SSHD 0.4.0 at > https://repository.apache.org/content/repositories/orgapachemina-003/ > > The release notes are available at > https://cwiki.apache.org/SSHD/sshd-040.html > > Please review and vote > > -- > Cheers, > Guillaume Nodet > > Blog: http://gnodet.blogspot.com/ > > Open Source SOA > http://fusesource.com > > > -- Cheers, Guillaume Nodet Blog: http://gnodet.blogspot.com/ Open Source SOA http://fusesource.com
[VOTE] Release SSHD 0.4.0 (2nd try)
I've uploaded a RC for SSHD 0.4.0 at https://repository.apache.org/content/repositories/orgapachemina-008/ The release notes are available at https://cwiki.apache.org/SSHD/sshd-040.html Please review and vote. -- Cheers, Guillaume Nodet Blog: http://gnodet.blogspot.com/ Open Source SOA http://fusesource.com
Re: Moving new nbxml in trunk
On Wed, May 5, 2010 at 10:53 AM, Bernd Fondermann wrote: > Probably the test coverage is not sufficient. That's a given for any code :-) As noted, we surely need more integration tests. > Did you actually try to connect a real client to a real server? Yes, I usually verify using Adium (which uses libpurple), Anyways, I reproduced the problem reported using Psi and fixed this and another bug (detecting XML stream restarts based on XML declarations). Psi can now connect. /niklas
[jira] Created: (VYSPER-202) Make Vysper report the correct version on Version-IQ request
Make Vysper report the correct version on Version-IQ request Key: VYSPER-202 URL: https://issues.apache.org/jira/browse/VYSPER-202 Project: VYSPER Issue Type: Bug Components: extension Affects Versions: 0.6 Reporter: Bernd Fondermann Priority: Blocker Just before making a new release, we need to code the right release number into SoftwareVersionIQHandler.VYSPER_RELEASE -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
On 5/5/10 11:52 AM, Guillaume Nodet wrote: I've uploaded a RC for SSHD 0.4.0 at https://repository.apache.org/content/repositories/orgapachemina-008/ The release notes are available at https://cwiki.apache.org/SSHD/sshd-040.html Please review and vote. +1 -- Regards, Cordialement, Emmanuel Lécharny www.nextury.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Is this SFTP fix checked into trunk or 0.4 tag? I do not see any changes to the SVN. Regards, Sai Pullabhotla On Wed, May 5, 2010 at 4:52 AM, Guillaume Nodet wrote: > I've uploaded a RC for SSHD 0.4.0 at > https://repository.apache.org/content/repositories/orgapachemina-008/ > > The release notes are available at > https://cwiki.apache.org/SSHD/sshd-040.html > > Please review and vote. > > -- > Cheers, > Guillaume Nodet > > Blog: http://gnodet.blogspot.com/ > > Open Source SOA > http://fusesource.com >
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Argh, I committed to my git repo and forgot to push the change. On Wed, May 5, 2010 at 14:53, Sai Pullabhotla wrote: > Is this SFTP fix checked into trunk or 0.4 tag? I do not see any > changes to the SVN. > > Regards, > Sai Pullabhotla > > > > > > On Wed, May 5, 2010 at 4:52 AM, Guillaume Nodet wrote: > > I've uploaded a RC for SSHD 0.4.0 at > > https://repository.apache.org/content/repositories/orgapachemina-008/ > > > > The release notes are available at > > https://cwiki.apache.org/SSHD/sshd-040.html > > > > Please review and vote. > > > > -- > > Cheers, > > Guillaume Nodet > > > > Blog: http://gnodet.blogspot.com/ > > > > Open Source SOA > > http://fusesource.com > > > -- Cheers, Guillaume Nodet Blog: http://gnodet.blogspot.com/ Open Source SOA http://fusesource.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Committed at rev 941273. Could you give it a try asap please ? I'll wait for your input before recutting the release. On Wed, May 5, 2010 at 15:02, Guillaume Nodet wrote: > Argh, I committed to my git repo and forgot to push the change. > > > On Wed, May 5, 2010 at 14:53, Sai Pullabhotla < > sai.pullabho...@jmethods.com> wrote: > >> Is this SFTP fix checked into trunk or 0.4 tag? I do not see any >> changes to the SVN. >> >> Regards, >> Sai Pullabhotla >> >> >> >> >> >> On Wed, May 5, 2010 at 4:52 AM, Guillaume Nodet wrote: >> > I've uploaded a RC for SSHD 0.4.0 at >> > https://repository.apache.org/content/repositories/orgapachemina-008/ >> > >> > The release notes are available at >> > https://cwiki.apache.org/SSHD/sshd-040.html >> > >> > Please review and vote. >> > >> > -- >> > Cheers, >> > Guillaume Nodet >> > >> > Blog: http://gnodet.blogspot.com/ >> > >> > Open Source SOA >> > http://fusesource.com >> > >> > > > > -- > Cheers, > Guillaume Nodet > > Blog: http://gnodet.blogspot.com/ > > Open Source SOA > http://fusesource.com > > > -- Cheers, Guillaume Nodet Blog: http://gnodet.blogspot.com/ Open Source SOA http://fusesource.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Okay, I got the fix and was able to see the directory listing using
FileZilla. However, a couple of issues:
1. Date/time on the files returned by the list are incorrect. Looks
like SSHD is always sending "Jan 01 00:00" for the date times.
2. I could not download or upload a file. Below is the trace info from
FileZilla. Basically, when I try to download a file, FileZilla is
requesting the incorrect path for what so ever reason. The path it
requests is
C:\Users\sai\opensource\sshd/C:\Users\sai\opensource\sshd\LICENSE.txt.
As you can see, it is prepending the working directory to the selected
file. I'm thinking it is due to the fact that we are sending "\" as
the path separator instead of "/".
Status: Connecting to localhost:...
Trace: Going to execute "C:\Program Files (x86)\FileZilla FTP
Client\fzsftp.exe"
Response: fzSftp started
Trace: CSftpControlSocket::ConnectParseResponse(fzSftp started)
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ConnectSend()
Command:keyfile "C:\Users\sai\temp\filezilla.ppk"
Trace: CSftpControlSocket::ConnectParseResponse()
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ConnectSend()
Command:open "p...@localhost"
Trace: Looking up host "localhost"
Trace: Connecting to 127.0.0.1 port
Trace: Server version: SSH-2.0-${POM.ARTIFACTID}-${POM.VERSION}
Trace: Using SSH protocol version 2
Trace: We claim version: SSH-2.0-PuTTY_Local:_Oct_10_2009_18:54:23
Trace: Using Diffie-Hellman with standard group "group14"
Trace: Doing Diffie-Hellman key exchange with hash SHA-1
Command:Trust new Hostkey: Once
Trace: Host key fingerprint is:
Trace: ssh-dss 1024 90:c8:10:01:dd:81:97:00:65:ec:c3:f9:d9:16:2d:b1
Trace: Initialised AES-256 CBC client->server encryption
Trace: Initialised HMAC-SHA1 client->server MAC algorithm
Trace: Initialised AES-256 CBC server->client encryption
Trace: Initialised HMAC-SHA1 server->client MAC algorithm
Trace: Successfully loaded 1 key pair from file
Trace: Offered public key from "C:\Users\sai\temp\filezilla.ppk"
Trace: Offer of public key accepted, trying to authenticate using it.
Trace: Access granted
Trace: Opened channel for session
Trace: Started a shell/command
Status: Connected to SAI
Trace: CSftpControlSocket::ConnectParseResponse()
Trace: CSftpControlSocket::ResetOperation(0)
Trace: CControlSocket::ResetOperation(0)
Status: Retrieving directory listing...
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ChangeDirSend()
Command:pwd
Response: Current directory is: "C:\Users\sai\opensource\sshd"
Trace: CSftpControlSocket::ResetOperation(0)
Trace: CControlSocket::ResetOperation(0)
Trace: CSftpControlSocket::ParseSubcommandResult(0)
Trace: CSftpControlSocket::ListSubcommandResult()
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ListSend()
Command:ls
Status: Listing directory C:\Users\sai\opensource\sshd
Listing:-rw-rw-rw- 1 psai psai 734 Jan 01 00:00
.classpath
Listing:-rw-rw-rw- 1 psai psai 55 Jan 01 00:00
.gitignore
Listing:-rw-rw-rw- 1 psai psai 558 Jan 01 00:00 .project
Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00 .settings
Listing:drw-rw-rw- 1 psai psai 4096 Jan 01 00:00 .svn
Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00 assembly
Listing:drw-rw-rw- 1 psai psai 4096 Jan 01 00:00 classes
Listing:-rw-rw-rw- 1 psai psai 680 Jan 01 00:00 key.pem
Listing:-rw-rw-rw- 1 psai psai11359 Jan 01 00:00
LICENSE.txt
Listing:-rw-rw-rw- 1 psai psai 3090 Jan 01 00:00
NOTICE.txt
Listing:-rw-rw-rw- 1 psai psai 8786 Jan 01 00:00 pom.xml
Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00 src
Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00 sshd-core
Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00 sshd-pam
Trace: CSftpControlSocket::ListParseResponse()
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ListSend()
Status: Calculating timezone offset of server...
Command:mtime ".classpath"
Error: get attrs for C:\Users\sai\opensource\sshd\.classpath: mtime not
provided
Trace: CSftpControlSocket::ListParseResponse()
Trace: CSftpControlSocket::ResetOperation(0)
Trace: CControlSocket::ResetOperation(0)
Status: Directory listing successful
Status: Connecting to localhost:...
Trace: Going to execute "C:\Program Files (x86)\FileZilla FTP
Client\fzsftp.exe"
Response: fzSftp started
Trace: CSftpControlSocket::ConnectParseResponse(fzSftp started)
Trace: CSftpControlSocket::SendNextCommand()
Trace: CSftpControlSocket::ConnectSend()
Command:keyfile "C:\Users\sai\temp\filezilla.ppk"
Trace: CSftpControlSocket::ConnectPa
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
I think I got it to work a little better by changing the code in
SftpSubsystem.sendAbsoluteName to as follows:
line 743:
buffer.putString("/" + file.getPath().replace('\\', '/'));
In order to make it work correctly on *NIX systems, we might need a
check to determine whether or not we should prepend another "/" to the
path. Also, the currently checked in code puts the path into the
buffer twice. I'm not sure if that is needed.
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 8:18 AM, Sai Pullabhotla
wrote:
> Okay, I got the fix and was able to see the directory listing using
> FileZilla. However, a couple of issues:
>
> 1. Date/time on the files returned by the list are incorrect. Looks
> like SSHD is always sending "Jan 01 00:00" for the date times.
> 2. I could not download or upload a file. Below is the trace info from
> FileZilla. Basically, when I try to download a file, FileZilla is
> requesting the incorrect path for what so ever reason. The path it
> requests is
> C:\Users\sai\opensource\sshd/C:\Users\sai\opensource\sshd\LICENSE.txt.
> As you can see, it is prepending the working directory to the selected
> file. I'm thinking it is due to the fact that we are sending "\" as
> the path separator instead of "/".
>
> Status: Connecting to localhost:...
> Trace: Going to execute "C:\Program Files (x86)\FileZilla FTP
> Client\fzsftp.exe"
> Response: fzSftp started
> Trace: CSftpControlSocket::ConnectParseResponse(fzSftp started)
> Trace: CSftpControlSocket::SendNextCommand()
> Trace: CSftpControlSocket::ConnectSend()
> Command: keyfile "C:\Users\sai\temp\filezilla.ppk"
> Trace: CSftpControlSocket::ConnectParseResponse()
> Trace: CSftpControlSocket::SendNextCommand()
> Trace: CSftpControlSocket::ConnectSend()
> Command: open "p...@localhost"
> Trace: Looking up host "localhost"
> Trace: Connecting to 127.0.0.1 port
> Trace: Server version: SSH-2.0-${POM.ARTIFACTID}-${POM.VERSION}
> Trace: Using SSH protocol version 2
> Trace: We claim version: SSH-2.0-PuTTY_Local:_Oct_10_2009_18:54:23
> Trace: Using Diffie-Hellman with standard group "group14"
> Trace: Doing Diffie-Hellman key exchange with hash SHA-1
> Command: Trust new Hostkey: Once
> Trace: Host key fingerprint is:
> Trace: ssh-dss 1024 90:c8:10:01:dd:81:97:00:65:ec:c3:f9:d9:16:2d:b1
> Trace: Initialised AES-256 CBC client->server encryption
> Trace: Initialised HMAC-SHA1 client->server MAC algorithm
> Trace: Initialised AES-256 CBC server->client encryption
> Trace: Initialised HMAC-SHA1 server->client MAC algorithm
> Trace: Successfully loaded 1 key pair from file
> Trace: Offered public key from "C:\Users\sai\temp\filezilla.ppk"
> Trace: Offer of public key accepted, trying to authenticate using it.
> Trace: Access granted
> Trace: Opened channel for session
> Trace: Started a shell/command
> Status: Connected to SAI
> Trace: CSftpControlSocket::ConnectParseResponse()
> Trace: CSftpControlSocket::ResetOperation(0)
> Trace: CControlSocket::ResetOperation(0)
> Status: Retrieving directory listing...
> Trace: CSftpControlSocket::SendNextCommand()
> Trace: CSftpControlSocket::ChangeDirSend()
> Command: pwd
> Response: Current directory is: "C:\Users\sai\opensource\sshd"
> Trace: CSftpControlSocket::ResetOperation(0)
> Trace: CControlSocket::ResetOperation(0)
> Trace: CSftpControlSocket::ParseSubcommandResult(0)
> Trace: CSftpControlSocket::ListSubcommandResult()
> Trace: CSftpControlSocket::SendNextCommand()
> Trace: CSftpControlSocket::ListSend()
> Command: ls
> Status: Listing directory C:\Users\sai\opensource\sshd
> Listing: -rw-rw-rw- 1 psai psai 734 Jan 01 00:00
> .classpath
> Listing: -rw-rw-rw- 1 psai psai 55 Jan 01 00:00
> .gitignore
> Listing: -rw-rw-rw- 1 psai psai 558 Jan 01 00:00
> .project
> Listing: drw-rw-rw- 1 psai psai 0 Jan 01 00:00
> .settings
> Listing: drw-rw-rw- 1 psai psai 4096 Jan 01 00:00 .svn
> Listing: drw-rw-rw- 1 psai psai 0 Jan 01 00:00
> assembly
> Listing: drw-rw-rw- 1 psai psai 4096 Jan 01 00:00 classes
> Listing: -rw-rw-rw- 1 psai psai 680 Jan 01 00:00 key.pem
> Listing: -rw-rw-rw- 1 psai psai 11359 Jan 01 00:00
> LICENSE.txt
> Listing: -rw-rw-rw- 1 psai psai 3090 Jan 01 00:00
> NOTICE.txt
> Listing: -rw-rw-rw- 1 psai psai 8786 Jan 01 00:00 pom.xml
> Listing: drw-rw-rw- 1 psai psai 0 Jan 01 00:00 src
> Listing: drw-rw-rw- 1 psai psai 0 Jan 01 00:00
> sshd-core
> Listing: drw-rw-rw- 1 psai psai 0 Jan 01 00:00
> sshd-pam
> Trace: CSftpControlSocket::ListParseResponse()
> Trace: CSftpControlSocket::SendNextCommand()
> Trace: CSftpControlSocket::ListSend()
> Status: Calculating timezone o
[jira] Commented: (VYSPER-202) Make Vysper report the correct version on Version-IQ request
[ https://issues.apache.org/jira/browse/VYSPER-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12864329#action_12864329 ] Niklas Gustavsson commented on VYSPER-202: -- How about we read this value from a properties file? If so, this can be automated by our Maven build in a similar way as we do in FtpServer. > Make Vysper report the correct version on Version-IQ request > > > Key: VYSPER-202 > URL: https://issues.apache.org/jira/browse/VYSPER-202 > Project: VYSPER > Issue Type: Bug > Components: extension >Affects Versions: 0.6 >Reporter: Bernd Fondermann >Priority: Blocker > > Just before making a new release, we need to code the right release number > into > SoftwareVersionIQHandler.VYSPER_RELEASE -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
On Wed, May 5, 2010 at 15:33, Sai Pullabhotla
wrote:
> I think I got it to work a little better by changing the code in
> SftpSubsystem.sendAbsoluteName to as follows:
>
> line 743:
>
>buffer.putString("/" + file.getPath().replace('\\', '/'));
>
>
Let my try something like that.
> In order to make it work correctly on *NIX systems, we might need a
> check to determine whether or not we should prepend another "/" to the
> path. Also, the currently checked in code puts the path into the
> buffer twice. I'm not sure if that is needed.
>
>
It is needed, but no idea why. You can try to remove it, but i had errors
when doing so. Any explanation is welcomed !
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 8:18 AM, Sai Pullabhotla
> wrote:
> > Okay, I got the fix and was able to see the directory listing using
> > FileZilla. However, a couple of issues:
> >
> > 1. Date/time on the files returned by the list are incorrect. Looks
> > like SSHD is always sending "Jan 01 00:00" for the date times.
> > 2. I could not download or upload a file. Below is the trace info from
> > FileZilla. Basically, when I try to download a file, FileZilla is
> > requesting the incorrect path for what so ever reason. The path it
> > requests is
> C:\Users\sai\opensource\sshd/C:\Users\sai\opensource\sshd\LICENSE.txt.
> > As you can see, it is prepending the working directory to the selected
> > file. I'm thinking it is due to the fact that we are sending "\" as
> > the path separator instead of "/".
> >
> > Status: Connecting to localhost:...
> > Trace: Going to execute "C:\Program Files (x86)\FileZilla FTP
> Client\fzsftp.exe"
> > Response: fzSftp started
> > Trace: CSftpControlSocket::ConnectParseResponse(fzSftp started)
> > Trace: CSftpControlSocket::SendNextCommand()
> > Trace: CSftpControlSocket::ConnectSend()
> > Command:keyfile "C:\Users\sai\temp\filezilla.ppk"
> > Trace: CSftpControlSocket::ConnectParseResponse()
> > Trace: CSftpControlSocket::SendNextCommand()
> > Trace: CSftpControlSocket::ConnectSend()
> > Command:open "p...@localhost"
> > Trace: Looking up host "localhost"
> > Trace: Connecting to 127.0.0.1 port
> > Trace: Server version: SSH-2.0-${POM.ARTIFACTID}-${POM.VERSION}
> > Trace: Using SSH protocol version 2
> > Trace: We claim version: SSH-2.0-PuTTY_Local:_Oct_10_2009_18:54:23
> > Trace: Using Diffie-Hellman with standard group "group14"
> > Trace: Doing Diffie-Hellman key exchange with hash SHA-1
> > Command:Trust new Hostkey: Once
> > Trace: Host key fingerprint is:
> > Trace: ssh-dss 1024 90:c8:10:01:dd:81:97:00:65:ec:c3:f9:d9:16:2d:b1
> > Trace: Initialised AES-256 CBC client->server encryption
> > Trace: Initialised HMAC-SHA1 client->server MAC algorithm
> > Trace: Initialised AES-256 CBC server->client encryption
> > Trace: Initialised HMAC-SHA1 server->client MAC algorithm
> > Trace: Successfully loaded 1 key pair from file
> > Trace: Offered public key from "C:\Users\sai\temp\filezilla.ppk"
> > Trace: Offer of public key accepted, trying to authenticate using it.
> > Trace: Access granted
> > Trace: Opened channel for session
> > Trace: Started a shell/command
> > Status: Connected to SAI
> > Trace: CSftpControlSocket::ConnectParseResponse()
> > Trace: CSftpControlSocket::ResetOperation(0)
> > Trace: CControlSocket::ResetOperation(0)
> > Status: Retrieving directory listing...
> > Trace: CSftpControlSocket::SendNextCommand()
> > Trace: CSftpControlSocket::ChangeDirSend()
> > Command:pwd
> > Response: Current directory is: "C:\Users\sai\opensource\sshd"
> > Trace: CSftpControlSocket::ResetOperation(0)
> > Trace: CControlSocket::ResetOperation(0)
> > Trace: CSftpControlSocket::ParseSubcommandResult(0)
> > Trace: CSftpControlSocket::ListSubcommandResult()
> > Trace: CSftpControlSocket::SendNextCommand()
> > Trace: CSftpControlSocket::ListSend()
> > Command:ls
> > Status: Listing directory C:\Users\sai\opensource\sshd
> > Listing:-rw-rw-rw- 1 psai psai 734 Jan 01 00:00
> .classpath
> > Listing:-rw-rw-rw- 1 psai psai 55 Jan 01 00:00
> .gitignore
> > Listing:-rw-rw-rw- 1 psai psai 558 Jan 01 00:00
> .project
> > Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00
> .settings
> > Listing:drw-rw-rw- 1 psai psai 4096 Jan 01 00:00
> .svn
> > Listing:drw-rw-rw- 1 psai psai0 Jan 01 00:00
> assembly
> > Listing:drw-rw-rw- 1 psai psai 4096 Jan 01 00:00
> classes
> > Listing:-rw-rw-rw- 1 psai psai 680 Jan 01 00:00
> key.pem
> > Listing:-rw-rw-rw- 1 psai psai11359 Jan 01 00:00
> LICENSE.txt
> > Listing:-rw-rw-rw- 1 psai psai 3090 Jan 01 00:00
> NOTICE.txt
> > Listing:-rw-rw-rw- 1 psai psai 8786 Jan 01 00:00
> pom.xml
> > Listing:drw-rw-r
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
I removed the second call to place the path in the buffer and seems to be working fine. Regards, Sai Pullabhotla
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
How about something like this:
protected void sendAbsoluteName(int id, File file) throws IOException {
Buffer buffer = new Buffer();
buffer.putByte((byte) SSH_FXP_NAME);
buffer.putInt(id);
buffer.putInt(1);
String path = file.getPath();
if(File.separatorChar != '/') {
path = path.replace(File.separatorChar, '/');
}
if(path.charAt(0) != '/') {
path = '/' + path;
}
buffer.putString(path);
//buffer.putString(file.getPath());
writeAttrs(buffer, file);
send(buffer);
}
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
wrote:
> I removed the second call to place the path in the buffer and seems to
> be working fine.
>
> Regards,
> Sai Pullabhotla
>
[jira] Commented: (VYSPER-202) Make Vysper report the correct version on Version-IQ request
[ https://issues.apache.org/jira/browse/VYSPER-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12864335#action_12864335 ] Bernd Fondermann commented on VYSPER-202: - +1, that's exactly what i missed to ask. if that's possible we wouldn't have to re-open this issue for every new release :) > Make Vysper report the correct version on Version-IQ request > > > Key: VYSPER-202 > URL: https://issues.apache.org/jira/browse/VYSPER-202 > Project: VYSPER > Issue Type: Bug > Components: extension >Affects Versions: 0.6 >Reporter: Bernd Fondermann >Priority: Blocker > > Just before making a new release, we need to code the right release number > into > SoftwareVersionIQHandler.VYSPER_RELEASE -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Yeah, that's what I'm trying.
Let me try with removing the double name though.
On Wed, May 5, 2010 at 15:50, Sai Pullabhotla
wrote:
> How about something like this:
>
>protected void sendAbsoluteName(int id, File file) throws IOException {
>Buffer buffer = new Buffer();
>buffer.putByte((byte) SSH_FXP_NAME);
>buffer.putInt(id);
>buffer.putInt(1);
>String path = file.getPath();
>if(File.separatorChar != '/') {
>path = path.replace(File.separatorChar, '/');
>}
>if(path.charAt(0) != '/') {
>path = '/' + path;
>}
>buffer.putString(path);
>//buffer.putString(file.getPath());
>writeAttrs(buffer, file);
>send(buffer);
>}
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> wrote:
> > I removed the second call to place the path in the buffer and seems to
> > be working fine.
> >
> > Regards,
> > Sai Pullabhotla
> >
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: Moving new nbxml in trunk
Niklas Gustavsson wrote: > > :-D Anyways, could you please verify the problem you had with MUC and > default namespaces now? Yes, I can verify that the problem has come back. (Is that the right answer, or the wrong one?) Bernd
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Removing the second string works with FileZilla but fails with the sftp
command line utility.
On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
> Yeah, that's what I'm trying.
> Let me try with removing the double name though.
>
>
> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
> sai.pullabho...@jmethods.com> wrote:
>
>> How about something like this:
>>
>>protected void sendAbsoluteName(int id, File file) throws IOException {
>>Buffer buffer = new Buffer();
>>buffer.putByte((byte) SSH_FXP_NAME);
>>buffer.putInt(id);
>>buffer.putInt(1);
>>String path = file.getPath();
>>if(File.separatorChar != '/') {
>>path = path.replace(File.separatorChar, '/');
>>}
>>if(path.charAt(0) != '/') {
>>path = '/' + path;
>>}
>>buffer.putString(path);
>>//buffer.putString(file.getPath());
>>writeAttrs(buffer, file);
>>send(buffer);
>>}
>>
>> Regards,
>> Sai Pullabhotla
>>
>>
>>
>>
>>
>> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
>> wrote:
>> > I removed the second call to place the path in the buffer and seems to
>> > be working fine.
>> >
>> > Regards,
>> > Sai Pullabhotla
>> >
>>
>
>
>
> --
> Cheers,
> Guillaume Nodet
>
> Blog: http://gnodet.blogspot.com/
>
> Open Source SOA
> http://fusesource.com
>
>
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
I've committed the changes. Could you give a try ?
On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
> Removing the second string works with FileZilla but fails with the sftp
> command line utility.
>
>
> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
>
>> Yeah, that's what I'm trying.
>> Let me try with removing the double name though.
>>
>>
>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
>> sai.pullabho...@jmethods.com> wrote:
>>
>>> How about something like this:
>>>
>>>protected void sendAbsoluteName(int id, File file) throws IOException
>>> {
>>>Buffer buffer = new Buffer();
>>>buffer.putByte((byte) SSH_FXP_NAME);
>>>buffer.putInt(id);
>>>buffer.putInt(1);
>>>String path = file.getPath();
>>>if(File.separatorChar != '/') {
>>>path = path.replace(File.separatorChar, '/');
>>>}
>>>if(path.charAt(0) != '/') {
>>>path = '/' + path;
>>>}
>>>buffer.putString(path);
>>>//buffer.putString(file.getPath());
>>>writeAttrs(buffer, file);
>>>send(buffer);
>>>}
>>>
>>> Regards,
>>> Sai Pullabhotla
>>>
>>>
>>>
>>>
>>>
>>> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
>>> wrote:
>>> > I removed the second call to place the path in the buffer and seems to
>>> > be working fine.
>>> >
>>> > Regards,
>>> > Sai Pullabhotla
>>> >
>>>
>>
>>
>>
>> --
>> Cheers,
>> Guillaume Nodet
>>
>> Blog: http://gnodet.blogspot.com/
>>
>> Open Source SOA
>> http://fusesource.com
>>
>>
>>
>
>
> --
> Cheers,
> Guillaume Nodet
>
> Blog: http://gnodet.blogspot.com/
>
> Open Source SOA
> http://fusesource.com
>
>
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: Moving new nbxml in trunk
Bernd Fondermann wrote: > Niklas Gustavsson wrote: >> :-D Anyways, could you please verify the problem you had with MUC and >> default namespaces now? > > Yes, I can verify that the problem has come back. > (Is that the right answer, or the wrong one?) ... and the known patch fixes it. Bernd
[CANCEL] [VOTE] Release SSHD 0.4.0 (2nd try)
Btw, I also cancel this vote. I'll upload a new candidate once everyone is happy ;-) On Wed, May 5, 2010 at 11:52, Guillaume Nodet wrote: > I've uploaded a RC for SSHD 0.4.0 at > https://repository.apache.org/content/repositories/orgapachemina-008/ > > The release notes are available at > https://cwiki.apache.org/SSHD/sshd-040.html > > Please review and vote. > > -- > Cheers, > Guillaume Nodet > > Blog: http://gnodet.blogspot.com/ > > Open Source SOA > http://fusesource.com > > > -- Cheers, Guillaume Nodet Blog: http://gnodet.blogspot.com/ Open Source SOA http://fusesource.com
[jira] Commented: (VYSPER-202) Make Vysper report the correct version on Version-IQ request
[ https://issues.apache.org/jira/browse/VYSPER-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12864346#action_12864346 ] Niklas Gustavsson commented on VYSPER-202: -- If you take care of reading for the properties file and reporting the results in Vysper, I can set up the build stuff. > Make Vysper report the correct version on Version-IQ request > > > Key: VYSPER-202 > URL: https://issues.apache.org/jira/browse/VYSPER-202 > Project: VYSPER > Issue Type: Bug > Components: extension >Affects Versions: 0.6 >Reporter: Bernd Fondermann >Priority: Blocker > > Just before making a new release, we need to code the right release number > into > SoftwareVersionIQHandler.VYSPER_RELEASE -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (VYSPER-202) Make Vysper report the correct version on Version-IQ request
[ https://issues.apache.org/jira/browse/VYSPER-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12864348#action_12864348 ] Bernd Fondermann commented on VYSPER-202: - deal. > Make Vysper report the correct version on Version-IQ request > > > Key: VYSPER-202 > URL: https://issues.apache.org/jira/browse/VYSPER-202 > Project: VYSPER > Issue Type: Bug > Components: extension >Affects Versions: 0.6 >Reporter: Bernd Fondermann >Priority: Blocker > > Just before making a new release, we need to code the right release number > into > SoftwareVersionIQHandler.VYSPER_RELEASE -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
I just noticed that the code checked in just replaces "\" with "/". I
think for best compliance on all operating systems and file systems,
it should replace File.separatorChar with "/". Not sure if there are
any file systems that use a separator other than "/" and "\".
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet wrote:
> I've committed the changes. Could you give a try ?
>
> On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
>
>> Removing the second string works with FileZilla but fails with the sftp
>> command line utility.
>>
>>
>> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
>>
>>> Yeah, that's what I'm trying.
>>> Let me try with removing the double name though.
>>>
>>>
>>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
>>> sai.pullabho...@jmethods.com> wrote:
>>>
How about something like this:
protected void sendAbsoluteName(int id, File file) throws IOException
{
Buffer buffer = new Buffer();
buffer.putByte((byte) SSH_FXP_NAME);
buffer.putInt(id);
buffer.putInt(1);
String path = file.getPath();
if(File.separatorChar != '/') {
path = path.replace(File.separatorChar, '/');
}
if(path.charAt(0) != '/') {
path = '/' + path;
}
buffer.putString(path);
//buffer.putString(file.getPath());
writeAttrs(buffer, file);
send(buffer);
}
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
wrote:
> I removed the second call to place the path in the buffer and seems to
> be working fine.
>
> Regards,
> Sai Pullabhotla
>
>>>
>>>
>>>
>>> --
>>> Cheers,
>>> Guillaume Nodet
>>>
>>> Blog: http://gnodet.blogspot.com/
>>>
>>> Open Source SOA
>>> http://fusesource.com
>>>
>>>
>>>
>>
>>
>> --
>> Cheers,
>> Guillaume Nodet
>>
>> Blog: http://gnodet.blogspot.com/
>>
>> Open Source SOA
>> http://fusesource.com
>>
>>
>>
>
>
> --
> Cheers,
> Guillaume Nodet
>
> Blog: http://gnodet.blogspot.com/
>
> Open Source SOA
> http://fusesource.com
>
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Haven't seen any such system, but i can easily change that, it will be
cleaner.
On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
wrote:
> I just noticed that the code checked in just replaces "\" with "/". I
> think for best compliance on all operating systems and file systems,
> it should replace File.separatorChar with "/". Not sure if there are
> any file systems that use a separator other than "/" and "\".
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet wrote:
> > I've committed the changes. Could you give a try ?
> >
> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
> >
> >> Removing the second string works with FileZilla but fails with the sftp
> >> command line utility.
> >>
> >>
> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
> >>
> >>> Yeah, that's what I'm trying.
> >>> Let me try with removing the double name though.
> >>>
> >>>
> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
> >>> sai.pullabho...@jmethods.com> wrote:
> >>>
> How about something like this:
>
> protected void sendAbsoluteName(int id, File file) throws
> IOException
> {
> Buffer buffer = new Buffer();
> buffer.putByte((byte) SSH_FXP_NAME);
> buffer.putInt(id);
> buffer.putInt(1);
> String path = file.getPath();
> if(File.separatorChar != '/') {
> path = path.replace(File.separatorChar, '/');
> }
> if(path.charAt(0) != '/') {
> path = '/' + path;
> }
> buffer.putString(path);
> //buffer.putString(file.getPath());
> writeAttrs(buffer, file);
> send(buffer);
> }
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> wrote:
> > I removed the second call to place the path in the buffer and seems
> to
> > be working fine.
> >
> > Regards,
> > Sai Pullabhotla
> >
>
> >>>
> >>>
> >>>
> >>> --
> >>> Cheers,
> >>> Guillaume Nodet
> >>>
> >>> Blog: http://gnodet.blogspot.com/
> >>>
> >>> Open Source SOA
> >>> http://fusesource.com
> >>>
> >>>
> >>>
> >>
> >>
> >> --
> >> Cheers,
> >> Guillaume Nodet
> >>
> >> Blog: http://gnodet.blogspot.com/
> >>
> >> Open Source SOA
> >> http://fusesource.com
> >>
> >>
> >>
> >
> >
> > --
> > Cheers,
> > Guillaume Nodet
> >
> > Blog: http://gnodet.blogspot.com/
> >
> > Open Source SOA
> > http://fusesource.com
> >
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Fixed, let me know if you find any other easily fixed issues.
On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
wrote:
> I just noticed that the code checked in just replaces "\" with "/". I
> think for best compliance on all operating systems and file systems,
> it should replace File.separatorChar with "/". Not sure if there are
> any file systems that use a separator other than "/" and "\".
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet wrote:
> > I've committed the changes. Could you give a try ?
> >
> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
> >
> >> Removing the second string works with FileZilla but fails with the sftp
> >> command line utility.
> >>
> >>
> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
> >>
> >>> Yeah, that's what I'm trying.
> >>> Let me try with removing the double name though.
> >>>
> >>>
> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
> >>> sai.pullabho...@jmethods.com> wrote:
> >>>
> How about something like this:
>
> protected void sendAbsoluteName(int id, File file) throws
> IOException
> {
> Buffer buffer = new Buffer();
> buffer.putByte((byte) SSH_FXP_NAME);
> buffer.putInt(id);
> buffer.putInt(1);
> String path = file.getPath();
> if(File.separatorChar != '/') {
> path = path.replace(File.separatorChar, '/');
> }
> if(path.charAt(0) != '/') {
> path = '/' + path;
> }
> buffer.putString(path);
> //buffer.putString(file.getPath());
> writeAttrs(buffer, file);
> send(buffer);
> }
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> wrote:
> > I removed the second call to place the path in the buffer and seems
> to
> > be working fine.
> >
> > Regards,
> > Sai Pullabhotla
> >
>
> >>>
> >>>
> >>>
> >>> --
> >>> Cheers,
> >>> Guillaume Nodet
> >>>
> >>> Blog: http://gnodet.blogspot.com/
> >>>
> >>> Open Source SOA
> >>> http://fusesource.com
> >>>
> >>>
> >>>
> >>
> >>
> >> --
> >> Cheers,
> >> Guillaume Nodet
> >>
> >> Blog: http://gnodet.blogspot.com/
> >>
> >> Open Source SOA
> >> http://fusesource.com
> >>
> >>
> >>
> >
> >
> > --
> > Cheers,
> > Guillaume Nodet
> >
> > Blog: http://gnodet.blogspot.com/
> >
> > Open Source SOA
> > http://fusesource.com
> >
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: Moving new nbxml in trunk
On Wed, May 5, 2010 at 4:25 PM, Bernd Fondermann wrote: > Bernd Fondermann wrote: >> Niklas Gustavsson wrote: >>> :-D Anyways, could you please verify the problem you had with MUC and >>> default namespaces now? >> >> Yes, I can verify that the problem has come back. >> (Is that the right answer, or the wrong one?) > > ... and the known patch fixes it. So, if I understand correctly, with the current code in trunk you're seeing the problem again? And, the prio patch of not printing default namespaces still fixed this issue? I just tested with a group chat with Psi and Adium and could not reproduce the problem, so I just wanted to make sure I understood the problem here :-) /niklas
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
How about the dates on the files?
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet wrote:
> Fixed, let me know if you find any other easily fixed issues.
>
> On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
> wrote:
>
>> I just noticed that the code checked in just replaces "\" with "/". I
>> think for best compliance on all operating systems and file systems,
>> it should replace File.separatorChar with "/". Not sure if there are
>> any file systems that use a separator other than "/" and "\".
>>
>> Regards,
>> Sai Pullabhotla
>>
>>
>>
>>
>>
>> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet wrote:
>> > I've committed the changes. Could you give a try ?
>> >
>> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
>> >
>> >> Removing the second string works with FileZilla but fails with the sftp
>> >> command line utility.
>> >>
>> >>
>> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
>> >>
>> >>> Yeah, that's what I'm trying.
>> >>> Let me try with removing the double name though.
>> >>>
>> >>>
>> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
>> >>> sai.pullabho...@jmethods.com> wrote:
>> >>>
>> How about something like this:
>>
>> protected void sendAbsoluteName(int id, File file) throws
>> IOException
>> {
>> Buffer buffer = new Buffer();
>> buffer.putByte((byte) SSH_FXP_NAME);
>> buffer.putInt(id);
>> buffer.putInt(1);
>> String path = file.getPath();
>> if(File.separatorChar != '/') {
>> path = path.replace(File.separatorChar, '/');
>> }
>> if(path.charAt(0) != '/') {
>> path = '/' + path;
>> }
>> buffer.putString(path);
>> //buffer.putString(file.getPath());
>> writeAttrs(buffer, file);
>> send(buffer);
>> }
>>
>> Regards,
>> Sai Pullabhotla
>>
>>
>>
>>
>>
>> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
>> wrote:
>> > I removed the second call to place the path in the buffer and seems
>> to
>> > be working fine.
>> >
>> > Regards,
>> > Sai Pullabhotla
>> >
>>
>> >>>
>> >>>
>> >>>
>> >>> --
>> >>> Cheers,
>> >>> Guillaume Nodet
>> >>>
>> >>> Blog: http://gnodet.blogspot.com/
>> >>>
>> >>> Open Source SOA
>> >>> http://fusesource.com
>> >>>
>> >>>
>> >>>
>> >>
>> >>
>> >> --
>> >> Cheers,
>> >> Guillaume Nodet
>> >>
>> >> Blog: http://gnodet.blogspot.com/
>> >>
>> >> Open Source SOA
>> >> http://fusesource.com
>> >>
>> >>
>> >>
>> >
>> >
>> > --
>> > Cheers,
>> > Guillaume Nodet
>> >
>> > Blog: http://gnodet.blogspot.com/
>> >
>> > Open Source SOA
>> > http://fusesource.com
>> >
>>
>
>
>
> --
> Cheers,
> Guillaume Nodet
>
> Blog: http://gnodet.blogspot.com/
>
> Open Source SOA
> http://fusesource.com
>
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Oh, by the way, it works fine for me with just putting (sending) the
path just once in the buffer. Tried with native sftp client on Ubuntu
Linux as well as Putty's SFTP command line client, which is what
FileZilla uses internally. I wonder why that does not work for you.
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 10:49 AM, Sai Pullabhotla
wrote:
> How about the dates on the files?
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet wrote:
>> Fixed, let me know if you find any other easily fixed issues.
>>
>> On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
>> wrote:
>>
>>> I just noticed that the code checked in just replaces "\" with "/". I
>>> think for best compliance on all operating systems and file systems,
>>> it should replace File.separatorChar with "/". Not sure if there are
>>> any file systems that use a separator other than "/" and "\".
>>>
>>> Regards,
>>> Sai Pullabhotla
>>>
>>>
>>>
>>>
>>>
>>> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet wrote:
>>> > I've committed the changes. Could you give a try ?
>>> >
>>> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet wrote:
>>> >
>>> >> Removing the second string works with FileZilla but fails with the sftp
>>> >> command line utility.
>>> >>
>>> >>
>>> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet wrote:
>>> >>
>>> >>> Yeah, that's what I'm trying.
>>> >>> Let me try with removing the double name though.
>>> >>>
>>> >>>
>>> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
>>> >>> sai.pullabho...@jmethods.com> wrote:
>>> >>>
>>> How about something like this:
>>>
>>> protected void sendAbsoluteName(int id, File file) throws
>>> IOException
>>> {
>>> Buffer buffer = new Buffer();
>>> buffer.putByte((byte) SSH_FXP_NAME);
>>> buffer.putInt(id);
>>> buffer.putInt(1);
>>> String path = file.getPath();
>>> if(File.separatorChar != '/') {
>>> path = path.replace(File.separatorChar, '/');
>>> }
>>> if(path.charAt(0) != '/') {
>>> path = '/' + path;
>>> }
>>> buffer.putString(path);
>>> //buffer.putString(file.getPath());
>>> writeAttrs(buffer, file);
>>> send(buffer);
>>> }
>>>
>>> Regards,
>>> Sai Pullabhotla
>>>
>>>
>>>
>>>
>>>
>>> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
>>> wrote:
>>> > I removed the second call to place the path in the buffer and seems
>>> to
>>> > be working fine.
>>> >
>>> > Regards,
>>> > Sai Pullabhotla
>>> >
>>>
>>> >>>
>>> >>>
>>> >>>
>>> >>> --
>>> >>> Cheers,
>>> >>> Guillaume Nodet
>>> >>>
>>> >>> Blog: http://gnodet.blogspot.com/
>>> >>>
>>> >>> Open Source SOA
>>> >>> http://fusesource.com
>>> >>>
>>> >>>
>>> >>>
>>> >>
>>> >>
>>> >> --
>>> >> Cheers,
>>> >> Guillaume Nodet
>>> >>
>>> >> Blog: http://gnodet.blogspot.com/
>>> >>
>>> >> Open Source SOA
>>> >> http://fusesource.com
>>> >>
>>> >>
>>> >>
>>> >
>>> >
>>> > --
>>> > Cheers,
>>> > Guillaume Nodet
>>> >
>>> > Blog: http://gnodet.blogspot.com/
>>> >
>>> > Open Source SOA
>>> > http://fusesource.com
>>> >
>>>
>>
>>
>>
>> --
>> Cheers,
>> Guillaume Nodet
>>
>> Blog: http://gnodet.blogspot.com/
>>
>> Open Source SOA
>> http://fusesource.com
>>
>
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Which dates ? I've fixed the "Jan 01 ..." thing if that's why you're
referring to.
But using filezilla, there are still things that we don't support, but i've
been able to download directories recursively without problems.
On Wed, May 5, 2010 at 17:49, Sai Pullabhotla
wrote:
> How about the dates on the files?
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet wrote:
> > Fixed, let me know if you find any other easily fixed issues.
> >
> > On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
> > wrote:
> >
> >> I just noticed that the code checked in just replaces "\" with "/". I
> >> think for best compliance on all operating systems and file systems,
> >> it should replace File.separatorChar with "/". Not sure if there are
> >> any file systems that use a separator other than "/" and "\".
> >>
> >> Regards,
> >> Sai Pullabhotla
> >>
> >>
> >>
> >>
> >>
> >> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet
> wrote:
> >> > I've committed the changes. Could you give a try ?
> >> >
> >> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet
> wrote:
> >> >
> >> >> Removing the second string works with FileZilla but fails with the
> sftp
> >> >> command line utility.
> >> >>
> >> >>
> >> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet
> wrote:
> >> >>
> >> >>> Yeah, that's what I'm trying.
> >> >>> Let me try with removing the double name though.
> >> >>>
> >> >>>
> >> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
> >> >>> sai.pullabho...@jmethods.com> wrote:
> >> >>>
> >> How about something like this:
> >>
> >> protected void sendAbsoluteName(int id, File file) throws
> >> IOException
> >> {
> >> Buffer buffer = new Buffer();
> >> buffer.putByte((byte) SSH_FXP_NAME);
> >> buffer.putInt(id);
> >> buffer.putInt(1);
> >> String path = file.getPath();
> >> if(File.separatorChar != '/') {
> >> path = path.replace(File.separatorChar, '/');
> >> }
> >> if(path.charAt(0) != '/') {
> >> path = '/' + path;
> >> }
> >> buffer.putString(path);
> >> //buffer.putString(file.getPath());
> >> writeAttrs(buffer, file);
> >> send(buffer);
> >> }
> >>
> >> Regards,
> >> Sai Pullabhotla
> >>
> >>
> >>
> >>
> >>
> >> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> >> wrote:
> >> > I removed the second call to place the path in the buffer and
> seems
> >> to
> >> > be working fine.
> >> >
> >> > Regards,
> >> > Sai Pullabhotla
> >> >
> >>
> >> >>>
> >> >>>
> >> >>>
> >> >>> --
> >> >>> Cheers,
> >> >>> Guillaume Nodet
> >> >>>
> >> >>> Blog: http://gnodet.blogspot.com/
> >> >>>
> >> >>> Open Source SOA
> >> >>> http://fusesource.com
> >> >>>
> >> >>>
> >> >>>
> >> >>
> >> >>
> >> >> --
> >> >> Cheers,
> >> >> Guillaume Nodet
> >> >>
> >> >> Blog: http://gnodet.blogspot.com/
> >> >>
> >> >> Open Source SOA
> >> >> http://fusesource.com
> >> >>
> >> >>
> >> >>
> >> >
> >> >
> >> > --
> >> > Cheers,
> >> > Guillaume Nodet
> >> >
> >> > Blog: http://gnodet.blogspot.com/
> >> >
> >> > Open Source SOA
> >> > http://fusesource.com
> >> >
> >>
> >
> >
> >
> > --
> > Cheers,
> > Guillaume Nodet
> >
> > Blog: http://gnodet.blogspot.com/
> >
> > Open Source SOA
> > http://fusesource.com
> >
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Okay, I see the fix for the dates, but they are still incorrect when
the client interprets its. Couple of things -
I believe time should be on a 24 hour clock basis - so the format need
to be changed to HH instead of hh.
The listing is not compatible with UNIX style listing. So files older
than 6 months should show the year, but not the time. Files in the
last 6 months should show time, but not year. Unless SFTP has a
standard for the directory list format.
In essence, FileZilla shows a file that is modified in year 2006 as 2010.
There is some code in the FtpServer project that we may want to steal.
It is org.apache.ftpserver.util.DateUtils.getUnixDate(long).
Regards,
Sai Pullabhotla
On Wed, May 5, 2010 at 11:15 AM, Guillaume Nodet wrote:
> Which dates ? I've fixed the "Jan 01 ..." thing if that's why you're
> referring to.
> But using filezilla, there are still things that we don't support, but i've
> been able to download directories recursively without problems.
>
> On Wed, May 5, 2010 at 17:49, Sai Pullabhotla
> wrote:
>
>> How about the dates on the files?
>>
>> Regards,
>> Sai Pullabhotla
>>
>>
>>
>>
>>
>> On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet wrote:
>> > Fixed, let me know if you find any other easily fixed issues.
>> >
>> > On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
>> > wrote:
>> >
>> >> I just noticed that the code checked in just replaces "\" with "/". I
>> >> think for best compliance on all operating systems and file systems,
>> >> it should replace File.separatorChar with "/". Not sure if there are
>> >> any file systems that use a separator other than "/" and "\".
>> >>
>> >> Regards,
>> >> Sai Pullabhotla
>> >>
>> >>
>> >>
>> >>
>> >>
>> >> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet
>> wrote:
>> >> > I've committed the changes. Could you give a try ?
>> >> >
>> >> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet
>> wrote:
>> >> >
>> >> >> Removing the second string works with FileZilla but fails with the
>> sftp
>> >> >> command line utility.
>> >> >>
>> >> >>
>> >> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet
>> wrote:
>> >> >>
>> >> >>> Yeah, that's what I'm trying.
>> >> >>> Let me try with removing the double name though.
>> >> >>>
>> >> >>>
>> >> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
>> >> >>> sai.pullabho...@jmethods.com> wrote:
>> >> >>>
>> >> How about something like this:
>> >>
>> >> protected void sendAbsoluteName(int id, File file) throws
>> >> IOException
>> >> {
>> >> Buffer buffer = new Buffer();
>> >> buffer.putByte((byte) SSH_FXP_NAME);
>> >> buffer.putInt(id);
>> >> buffer.putInt(1);
>> >> String path = file.getPath();
>> >> if(File.separatorChar != '/') {
>> >> path = path.replace(File.separatorChar, '/');
>> >> }
>> >> if(path.charAt(0) != '/') {
>> >> path = '/' + path;
>> >> }
>> >> buffer.putString(path);
>> >> //buffer.putString(file.getPath());
>> >> writeAttrs(buffer, file);
>> >> send(buffer);
>> >> }
>> >>
>> >> Regards,
>> >> Sai Pullabhotla
>> >>
>> >>
>> >>
>> >>
>> >>
>> >> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
>> >> wrote:
>> >> > I removed the second call to place the path in the buffer and
>> seems
>> >> to
>> >> > be working fine.
>> >> >
>> >> > Regards,
>> >> > Sai Pullabhotla
>> >> >
>> >>
>> >> >>>
>> >> >>>
>> >> >>>
>> >> >>> --
>> >> >>> Cheers,
>> >> >>> Guillaume Nodet
>> >> >>>
>> >> >>> Blog: http://gnodet.blogspot.com/
>> >> >>>
>> >> >>> Open Source SOA
>> >> >>> http://fusesource.com
>> >> >>>
>> >> >>>
>> >> >>>
>> >> >>
>> >> >>
>> >> >> --
>> >> >> Cheers,
>> >> >> Guillaume Nodet
>> >> >>
>> >> >> Blog: http://gnodet.blogspot.com/
>> >> >>
>> >> >> Open Source SOA
>> >> >> http://fusesource.com
>> >> >>
>> >> >>
>> >> >>
>> >> >
>> >> >
>> >> > --
>> >> > Cheers,
>> >> > Guillaume Nodet
>> >> >
>> >> > Blog: http://gnodet.blogspot.com/
>> >> >
>> >> > Open Source SOA
>> >> > http://fusesource.com
>> >> >
>> >>
>> >
>> >
>> >
>> > --
>> > Cheers,
>> > Guillaume Nodet
>> >
>> > Blog: http://gnodet.blogspot.com/
>> >
>> > Open Source SOA
>> > http://fusesource.com
>> >
>>
>
>
>
> --
> Cheers,
> Guillaume Nodet
>
> Blog: http://gnodet.blogspot.com/
>
> Open Source SOA
> http://fusesource.com
>
Re: [VOTE] Release SSHD 0.4.0 (2nd try)
Gents,
Can we please change the topic of this email to a dev discussion... the [VOTE]
in it is probably not appropriate since [VOTE] indicates a real vote. Thanks!
Jeff
On May 5, 2010, at 10:43 AM, Sai Pullabhotla wrote:
> Okay, I see the fix for the dates, but they are still incorrect when
> the client interprets its. Couple of things -
>
> I believe time should be on a 24 hour clock basis - so the format need
> to be changed to HH instead of hh.
>
> The listing is not compatible with UNIX style listing. So files older
> than 6 months should show the year, but not the time. Files in the
> last 6 months should show time, but not year. Unless SFTP has a
> standard for the directory list format.
>
> In essence, FileZilla shows a file that is modified in year 2006 as 2010.
>
> There is some code in the FtpServer project that we may want to steal.
> It is org.apache.ftpserver.util.DateUtils.getUnixDate(long).
>
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 11:15 AM, Guillaume Nodet wrote:
>> Which dates ? I've fixed the "Jan 01 ..." thing if that's why you're
>> referring to.
>> But using filezilla, there are still things that we don't support, but i've
>> been able to download directories recursively without problems.
>>
>> On Wed, May 5, 2010 at 17:49, Sai Pullabhotla
>> wrote:
>>
>>> How about the dates on the files?
>>>
>>> Regards,
>>> Sai Pullabhotla
>>>
>>>
>>>
>>>
>>>
>>> On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet wrote:
Fixed, let me know if you find any other easily fixed issues.
On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
wrote:
> I just noticed that the code checked in just replaces "\" with "/". I
> think for best compliance on all operating systems and file systems,
> it should replace File.separatorChar with "/". Not sure if there are
> any file systems that use a separator other than "/" and "\".
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet
>>> wrote:
>> I've committed the changes. Could you give a try ?
>>
>> On Wed, May 5, 2010 at 16:20, Guillaume Nodet
>>> wrote:
>>
>>> Removing the second string works with FileZilla but fails with the
>>> sftp
>>> command line utility.
>>>
>>>
>>> On Wed, May 5, 2010 at 16:13, Guillaume Nodet
>>> wrote:
>>>
Yeah, that's what I'm trying.
Let me try with removing the double name though.
On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
sai.pullabho...@jmethods.com> wrote:
> How about something like this:
>
>protected void sendAbsoluteName(int id, File file) throws
> IOException
> {
>Buffer buffer = new Buffer();
>buffer.putByte((byte) SSH_FXP_NAME);
>buffer.putInt(id);
>buffer.putInt(1);
>String path = file.getPath();
>if(File.separatorChar != '/') {
>path = path.replace(File.separatorChar, '/');
>}
>if(path.charAt(0) != '/') {
>path = '/' + path;
>}
>buffer.putString(path);
>//buffer.putString(file.getPath());
>writeAttrs(buffer, file);
>send(buffer);
>}
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> wrote:
>> I removed the second call to place the path in the buffer and
>>> seems
> to
>> be working fine.
>>
>> Regards,
>> Sai Pullabhotla
>>
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
>>>
>>>
>>> --
>>> Cheers,
>>> Guillaume Nodet
>>>
>>> Blog: http://gnodet.blogspot.com/
>>>
>>> Open Source SOA
>>> http://fusesource.com
>>>
>>>
>>>
>>
>>
>> --
>> Cheers,
>> Guillaume Nodet
>>
>> Blog: http://gnodet.blogspot.com/
>>
>> Open Source SOA
>> http://fusesource.com
>>
>
--
Cheers,
Guillaume Nodet
Blog: http://gnodet.blogspot.com/
Open Source SOA
http://fusesource.com
>>>
>>
>>
>>
>> --
>> Cheers,
>> Guillaume Nodet
>>
>> Blog: http://gnodet.blogspot.com/
>>
>> Open Source SOA
>>
Re: Moving new nbxml in trunk
On Wed, May 5, 2010 at 17:25, Niklas Gustavsson wrote: > On Wed, May 5, 2010 at 4:25 PM, Bernd Fondermann > wrote: >> Bernd Fondermann wrote: >>> Niklas Gustavsson wrote: :-D Anyways, could you please verify the problem you had with MUC and default namespaces now? >>> >>> Yes, I can verify that the problem has come back. >>> (Is that the right answer, or the wrong one?) >> >> ... and the known patch fixes it. > > So, if I understand correctly, with the current code in trunk you're > seeing the problem again? And, the prio patch of not printing default > namespaces still fixed this issue? Correct. > I just tested with a group chat > with Psi and Adium and could not reproduce the problem, so I just > wanted to make sure I understood the problem here :-) The group chat problem is between Psi and Psi. But I didn't test that today, but will tomorrow. What I did test is checking if PubSubGUI worked. It uses smack. That's where I saw and cured the problem. And I don't think those xmlns="" should be there. Isn't this kind of overwriting the default namespace, when you really want the default namespace from the outer element to propagate? Bernd
SFTP problems
Thanks for the pointer. The SFTP subsystem is now using that code.
Any more issues ?
On Wed, May 5, 2010 at 18:43, Sai Pullabhotla
wrote:
> Okay, I see the fix for the dates, but they are still incorrect when
> the client interprets its. Couple of things -
>
> I believe time should be on a 24 hour clock basis - so the format need
> to be changed to HH instead of hh.
>
> The listing is not compatible with UNIX style listing. So files older
> than 6 months should show the year, but not the time. Files in the
> last 6 months should show time, but not year. Unless SFTP has a
> standard for the directory list format.
>
> In essence, FileZilla shows a file that is modified in year 2006 as 2010.
>
> There is some code in the FtpServer project that we may want to steal.
> It is org.apache.ftpserver.util.DateUtils.getUnixDate(long).
>
>
> Regards,
> Sai Pullabhotla
>
>
>
>
>
> On Wed, May 5, 2010 at 11:15 AM, Guillaume Nodet wrote:
> > Which dates ? I've fixed the "Jan 01 ..." thing if that's why you're
> > referring to.
> > But using filezilla, there are still things that we don't support, but
> i've
> > been able to download directories recursively without problems.
> >
> > On Wed, May 5, 2010 at 17:49, Sai Pullabhotla
> > wrote:
> >
> >> How about the dates on the files?
> >>
> >> Regards,
> >> Sai Pullabhotla
> >>
> >>
> >>
> >>
> >>
> >> On Wed, May 5, 2010 at 10:13 AM, Guillaume Nodet
> wrote:
> >> > Fixed, let me know if you find any other easily fixed issues.
> >> >
> >> > On Wed, May 5, 2010 at 16:41, Sai Pullabhotla
> >> > wrote:
> >> >
> >> >> I just noticed that the code checked in just replaces "\" with "/". I
> >> >> think for best compliance on all operating systems and file systems,
> >> >> it should replace File.separatorChar with "/". Not sure if there are
> >> >> any file systems that use a separator other than "/" and "\".
> >> >>
> >> >> Regards,
> >> >> Sai Pullabhotla
> >> >>
> >> >>
> >> >>
> >> >>
> >> >>
> >> >> On Wed, May 5, 2010 at 9:22 AM, Guillaume Nodet
> >> wrote:
> >> >> > I've committed the changes. Could you give a try ?
> >> >> >
> >> >> > On Wed, May 5, 2010 at 16:20, Guillaume Nodet
> >> wrote:
> >> >> >
> >> >> >> Removing the second string works with FileZilla but fails with the
> >> sftp
> >> >> >> command line utility.
> >> >> >>
> >> >> >>
> >> >> >> On Wed, May 5, 2010 at 16:13, Guillaume Nodet
> >> wrote:
> >> >> >>
> >> >> >>> Yeah, that's what I'm trying.
> >> >> >>> Let me try with removing the double name though.
> >> >> >>>
> >> >> >>>
> >> >> >>> On Wed, May 5, 2010 at 15:50, Sai Pullabhotla <
> >> >> >>> sai.pullabho...@jmethods.com> wrote:
> >> >> >>>
> >> >> How about something like this:
> >> >>
> >> >> protected void sendAbsoluteName(int id, File file) throws
> >> >> IOException
> >> >> {
> >> >> Buffer buffer = new Buffer();
> >> >> buffer.putByte((byte) SSH_FXP_NAME);
> >> >> buffer.putInt(id);
> >> >> buffer.putInt(1);
> >> >> String path = file.getPath();
> >> >> if(File.separatorChar != '/') {
> >> >> path = path.replace(File.separatorChar, '/');
> >> >> }
> >> >> if(path.charAt(0) != '/') {
> >> >> path = '/' + path;
> >> >> }
> >> >> buffer.putString(path);
> >> >> //buffer.putString(file.getPath());
> >> >> writeAttrs(buffer, file);
> >> >> send(buffer);
> >> >> }
> >> >>
> >> >> Regards,
> >> >> Sai Pullabhotla
> >> >>
> >> >>
> >> >>
> >> >>
> >> >>
> >> >> On Wed, May 5, 2010 at 8:45 AM, Sai Pullabhotla
> >> >> wrote:
> >> >> > I removed the second call to place the path in the buffer and
> >> seems
> >> >> to
> >> >> > be working fine.
> >> >> >
> >> >> > Regards,
> >> >> > Sai Pullabhotla
> >> >> >
> >> >>
> >> >> >>>
> >> >> >>>
> >> >> >>>
> >> >> >>> --
> >> >> >>> Cheers,
> >> >> >>> Guillaume Nodet
> >> >> >>>
> >> >> >>> Blog: http://gnodet.blogspot.com/
> >> >> >>>
> >> >> >>> Open Source SOA
> >> >> >>> http://fusesource.com
> >> >> >>>
> >> >> >>>
> >> >> >>>
> >> >> >>
> >> >> >>
> >> >> >> --
> >> >> >> Cheers,
> >> >> >> Guillaume Nodet
> >> >> >>
> >> >> >> Blog: http://gnodet.blogspot.com/
> >> >> >>
> >> >> >> Open Source SOA
> >> >> >> http://fusesource.com
> >> >> >>
> >> >> >>
> >> >> >>
> >> >> >
> >> >> >
> >> >> > --
> >> >> > Cheers,
> >> >> > Guillaume Nodet
> >> >> >
> >> >> > Blog: http://gnodet.blogspot.com/
> >> >> >
> >> >> > Open Source SOA
> >> >> > http://fusesource.com
> >> >> >
> >> >>
> >> >
> >> >
> >> >
> >> > --
> >> > Cheers,
> >> > Guillaume Nodet
> >> >
> >> > Blog: http://gnodet.blogspot.com/
> >> >
>
Re: Moving new nbxml in trunk
On Wed, May 5, 2010 at 7:55 PM, Bernd Fondermann wrote: > The group chat problem is between Psi and Psi. But I didn't test that > today, but will tomorrow. Great, let me know the results (or how to reproduce it). > And I don't think those xmlns="" should be there. Isn't this kind of > overwriting the default namespace, when you really want the default > namespace from the outer element to propagate? Right, it sets the default namespace to be the empty namespace. This is normally due to us not setting the correct namespace when creating elements. See rev 941337 for an example of fixing this problem. /niklas
