[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411695349 ## File path: sshd-core/src/main/java/org/apache/sshd/common/channel/ChannelAsyncOutputStream.java ## @@ -107,9 +108,39 @@ protected synchronized void doW

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411695054 ## File path: sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java ## @@ -954,7 +954,9 @@ protected IoWriteFuture doWritePa

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411694928 ## File path: sshd-sftp/pom.xml ## @@ -85,6 +85,18 @@ jzlib test + +org.testcontainers +

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411692176 ## File path: sshd-sftp/src/test/java/org/apache/sshd/client/subsystem/sftp/SftpTransferTest.java ## @@ -0,0 +1,131 @@ +/* + * Licensed to the Apache Soft

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411689843 ## File path: sshd-sftp/src/main/java/org/apache/sshd/client/subsystem/sftp/impl/SftpRemotePathChannel.java ## @@ -524,13 +527,13 @@ protected void endBlo

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411687971 ## File path: sshd-sftp/src/main/java/org/apache/sshd/client/subsystem/sftp/impl/SftpRemotePathChannel.java ## @@ -346,30 +355,19 @@ public long transferT

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411688147 ## File path: sshd-sftp/src/main/java/org/apache/sshd/client/subsystem/sftp/impl/SftpRemotePathChannel.java ## @@ -410,18 +408,23 @@ public long transferF

[GitHub] [mina-sshd] gnodet commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411682132 ## File path: sshd-sftp/src/main/java/org/apache/sshd/client/subsystem/sftp/impl/AbstractSftpClient.java ## @@ -783,10 +786,6 @@ public void rename(String

Re: Autoformat source code issue

2020-04-20 Thread Guillaume Nodet
The plugin is definitely supposed to handle all line endings kind. I've boot up a windows 10 VM, cloned the git repo, built the whole sshd project. modified: sshd-common/src/main/java/org/apache/sshd/common/cipher/package.html modified: sshd-common/src/main/java/org/apache/sshd/com

Re: Autoformat source code issue

2020-04-20 Thread Jonathan Valliere
Did you delete the whole repo and re-clone it? I have no idea, I don’t use Windows for anything relating to work. On Mon, Apr 20, 2020 at 1:49 PM Lyor Goldstein wrote: > >> >> > https://stackoverflow.com/questions/21822650/disable-git-eol-conversions > >> >> You will probably have to checkout

Re: Autoformat source code issue

2020-04-20 Thread Lyor Goldstein
>> >> https://stackoverflow.com/questions/21822650/disable-git-eol-conversions >> >> You will probably have to checkout the repo again. >> Thx - I'll give it a try... No good - tried all sort of combinations for .gitconfig and .gitattributes but none worked quite frustrating - makes no sense

[GitHub] [mina-sshd] FliegenKLATSCH commented on issue #122: Allow colon as valid host pattern char to support ipv6 entries in kno…

2020-04-20 Thread GitBox
FliegenKLATSCH commented on issue #122: URL: https://github.com/apache/mina-sshd/pull/122#issuecomment-616704666 Should be fine now. Do you already have any plannings regarding the next release (date)? This is an automated

Re: Autoformat source code issue

2020-04-20 Thread Lyor Goldstein
>> https://stackoverflow.com/questions/21822650/disable-git-eol-conversions >> You will probably have to checkout the repo again. Thx - I'll give it a try...

Re: Autoformat source code issue

2020-04-20 Thread Jonathan Valliere
https://stackoverflow.com/questions/21822650/disable-git-eol-conversions You will probably have to checkout the repo again. On Mon, Apr 20, 2020 at 1:23 PM Lyor Goldstein wrote: > >> Maven auto format might be converting it back to LF then GIT gets > confused? Maybe just disable the conversio

Re: Autoformat source code issue

2020-04-20 Thread Lyor Goldstein
>> Maven auto format might be converting it back to LF then GIT gets confused? Maybe just disable the conversion to CRLF? How do I do that ?

[GitHub] [mina-sshd] lgoldstein commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
lgoldstein commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411554606 ## File path: sshd-sftp/pom.xml ## @@ -85,6 +85,18 @@ jzlib test + +org.testcontainers +

[GitHub] [mina-sshd] bsideup commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
bsideup commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411495180 ## File path: sshd-sftp/pom.xml ## @@ -85,6 +85,18 @@ jzlib test + +org.testcontainers +

Re: Autoformat source code issue

2020-04-20 Thread Jonathan Valliere
Maven auto format might be converting it back to LF then GIT gets confused? Maybe just disable the conversion to CRLF? On Mon, Apr 20, 2020 at 11:03 AM Lyor Goldstein wrote: > >> I thought GIT does this automatically on Windows. > > Not entirely - there are some core settings that control it, b

Re: Autoformat source code issue

2020-04-20 Thread Lyor Goldstein
>> I thought GIT does this automatically on Windows. Not entirely - there are some core settings that control it, but it is not clear how they affect the behavior. My current setup in .gitconfig is shown below: [core] autocrlf = input filemode = false I have been using it so far wit

Re: Autoformat source code issue

2020-04-20 Thread Jonathan Valliere
I thought GIT does this automatically on Windows. On Mon, Apr 20, 2020 at 10:30 AM Lyor Goldstein wrote: > I have just fetched the latest master branch and built it on my Windows > machine and am getting as if all files have been changed - the change has > to do with LF -> CRLF change in all fi

[jira] [Commented] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread Marcin L (Jira)
[ https://issues.apache.org/jira/browse/DIRMINA-1122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087789#comment-17087789 ] Marcin L commented on DIRMINA-1122: --- Cool. You commented about the same time I submit

Autoformat source code issue

2020-04-20 Thread Lyor Goldstein
I have just fetched the latest master branch and built it on my Windows machine and am getting as if all files have been changed - the change has to do with LF -> CRLF change in all files. How do we prevent this from happening? Personally I prefer LF, but let's make sure that building on Linux or

[jira] [Commented] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread Jira
[ https://issues.apache.org/jira/browse/DIRMINA-1122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087782#comment-17087782 ] Emmanuel Lécharny commented on DIRMINA-1122: Actually, no, and I just check

[GitHub] [mina-sshd] FliegenKLATSCH commented on issue #119: Add support for openssh host key certificates

2020-04-20 Thread GitBox
FliegenKLATSCH commented on issue #119: URL: https://github.com/apache/mina-sshd/pull/119#issuecomment-616590045 Ok, thanks. Changelog might just be misleading. This is an automated message from the Apache Git Service. To res

[jira] [Resolved] (SSHD-660) Add support for authentication using signed client/server keys

2020-04-20 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lyor Goldstein resolved SSHD-660. - Fix Version/s: 2.4.1 Resolution: Fixed > Add support for authentication using signed client/

[GitHub] [mina-sshd] lgoldstein commented on a change in pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
lgoldstein commented on a change in pull request #123: URL: https://github.com/apache/mina-sshd/pull/123#discussion_r411387849 ## File path: sshd-core/src/main/java/org/apache/sshd/common/channel/ChannelAsyncOutputStream.java ## @@ -107,9 +108,39 @@ protected synchronized void

[jira] [Commented] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread Marcin L (Jira)
[ https://issues.apache.org/jira/browse/DIRMINA-1122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087771#comment-17087771 ] Marcin L commented on DIRMINA-1122: --- Yes - you probably can. The problem is to suppor

[jira] [Comment Edited] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread Jira
[ https://issues.apache.org/jira/browse/DIRMINA-1122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087766#comment-17087766 ] Emmanuel Lécharny edited comment on DIRMINA-1122 at 4/20/20, 2:08 PM: ---

[jira] [Commented] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread Jira
[ https://issues.apache.org/jira/browse/DIRMINA-1122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087766#comment-17087766 ] Emmanuel Lécharny commented on DIRMINA-1122: I believe this can be done thr

[jira] [Work logged] (DIRMINA-1122) Add support for endpoint identification algorithm

2020-04-20 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/DIRMINA-1122?focusedWorklogId=425327&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-425327 ] ASF GitHub Bot logged work on DIRMINA-1122: --- Author: ASF GitHub B

[GitHub] [mina] the-thing opened a new pull request #26: DIRMINA-1122 - added support for endpoint identification algorithm

2020-04-20 Thread GitBox
the-thing opened a new pull request #26: URL: https://github.com/apache/mina/pull/26 https://issues.apache.org/jira/projects/DIRMINA/issues/DIRMINA-1122 We found out that it is not possible to set endpoint identification algorithm while discussing SNI support in quickfixj https://gi

Re: [mina-sshd] 01/02: [SSHD-660] Add support for serer side openssh host certkeys

2020-04-20 Thread Guillaume Nodet
Yes, sorry for the noise. I must have used a not clean repo before building or maybe not rebuild from root, not sure. Anyway, I can confirm that it's working for me too. Le lun. 20 avr. 2020 à 15:37, Lyor Goldstein a écrit : > >> I think your forgot to commit the changes to a few files: KeyUti

[GitHub] [mina-sshd] lgoldstein commented on issue #122: Allow colon as valid host pattern char to support ipv6 entries in kno…

2020-04-20 Thread GitBox
lgoldstein commented on issue #122: URL: https://github.com/apache/mina-sshd/pull/122#issuecomment-616559748 Please resolve the conflicts and let me know when you feel it is ready to merge This is an automated message from t

Re: [mina-sshd] 01/02: [SSHD-660] Add support for serer side openssh host certkeys

2020-04-20 Thread Lyor Goldstein
>> I think your forgot to commit the changes to a few files: KeyUtils, KeyPairProvider I don't think so - I have used the cloned repository commits rather that the PR + the code compiles and passes all tests.

[GitHub] [mina-sshd] lgoldstein commented on issue #119: Add support for openssh host key certificates

2020-04-20 Thread GitBox
lgoldstein commented on issue #119: URL: https://github.com/apache/mina-sshd/pull/119#issuecomment-616557210 >> Thanks a lot, but it's not only server side support but also client side (I started with client side actually and had two different commits..). Sorry - too late now - the i

[GitHub] [mina-sshd] gnodet opened a new pull request #123: SFTP transfer speeds, clean API

2020-04-20 Thread GitBox
gnodet opened a new pull request #123: URL: https://github.com/apache/mina-sshd/pull/123 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [mina] snufkinsnorka commented on issue #25: make NioProcessor and NioSocketAcceptor extendable by removing final from class declaration

2020-04-20 Thread GitBox
snufkinsnorka commented on issue #25: URL: https://github.com/apache/mina/pull/25#issuecomment-616402730 We want to extend those in order to add ability of moving IoSession established sockets from processor to processor. We use this when we want to move connections between different se

[GitHub] [mina] elecharny commented on issue #25: make NioProcessor and NioSocketAcceptor extendable by removing final from class declaration

2020-04-20 Thread GitBox
elecharny commented on issue #25: URL: https://github.com/apache/mina/pull/25#issuecomment-616378510 Wondering: why would you make those classes extensible ? This is an automated message from the Apache Git Service. To respon

[GitHub] [mina] snufkinsnorka opened a new pull request #25: make NioProcessor and NioSocketAcceptor extendable by removing final from class declaration

2020-04-20 Thread GitBox
snufkinsnorka opened a new pull request #25: URL: https://github.com/apache/mina/pull/25 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g