[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466531=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466531 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[GitHub] [mina-sshd] tomaswolf commented on pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
tomaswolf commented on pull request #156: URL: https://github.com/apache/mina-sshd/pull/156#issuecomment-668882088 Sorry, that doesn't change anything. The following sequence is still possible: exceptionCaught: calls authFutureHolder.get(), gets the InitialAuthFuture auth: calls

[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466443=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466443 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[GitHub] [mina-sshd] lgoldstein commented on pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
lgoldstein commented on pull request #156: URL: https://github.com/apache/mina-sshd/pull/156#issuecomment-668817455 A bit complex to my taste - I have pushed a fix for my original commit - basically I am using an `AtomicReference` instead of `volatile` to manage the replacement of the

[GitHub] [mina-sshd] tomaswolf commented on pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
tomaswolf commented on pull request #156: URL: https://github.com/apache/mina-sshd/pull/156#issuecomment-668803523 I've pushed https://github.com/tomaswolf/mina-sshd/commit/3f15c6dbcce1dae0c88cae5b2ad0c2f07ebbc43e in https://github.com/tomaswolf/mina-sshd/tree/SSHD-1050 . It's a follow-up

[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466433=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466433 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466193=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466193 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[GitHub] [mina-sshd] lgoldstein commented on a change in pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
lgoldstein commented on a change in pull request #156: URL: https://github.com/apache/mina-sshd/pull/156#discussion_r465005694 ## File path: sshd-core/src/main/java/org/apache/sshd/client/session/ClientSessionImpl.java ## @@ -124,8 +123,19 @@ public AuthFuture auth() throws

[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466179=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466179 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[GitHub] [mina-sshd] tomaswolf commented on a change in pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
tomaswolf commented on a change in pull request #156: URL: https://github.com/apache/mina-sshd/pull/156#discussion_r464964829 ## File path: sshd-core/src/main/java/org/apache/sshd/client/session/ClientSessionImpl.java ## @@ -124,8 +123,19 @@ public AuthFuture auth() throws

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170642#comment-17170642 ] Lyor Goldstein commented on SSHD-1050: -- [~wolft] see https://github.com/apache/mina-sshd/pull/156 - I

[jira] [Work logged] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?focusedWorklogId=466094=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-466094 ] ASF GitHub Bot logged work on SSHD-1050: Author: ASF GitHub Bot

[GitHub] [mina-sshd] lgoldstein opened a new pull request #156: [SSHD-1050] Fixed race condition in AuthFuture if exception caught before authentication started

2020-08-04 Thread GitBox
lgoldstein opened a new pull request #156: URL: https://github.com/apache/mina-sshd/pull/156 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

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170616#comment-17170616 ] Lyor Goldstein commented on SSHD-1050: -- Actually, I think I have an idea - working on it - will let

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Thomas Wolf (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170615#comment-17170615 ] Thomas Wolf commented on SSHD-1050: --- {quote}If an exception occurs while the old instance is still in

[jira] [Assigned] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lyor Goldstein reassigned SSHD-1050: Assignee: Lyor Goldstein > Race condition between early exceptions and AuthFuture >

[jira] [Work started] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on SSHD-1050 started by Lyor Goldstein. > Race condition between early exceptions and AuthFuture >

[jira] [Comment Edited] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170600#comment-17170600 ] Lyor Goldstein edited comment on SSHD-1050 at 8/4/20, 7:13 AM: --- Found it -

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170600#comment-17170600 ] Lyor Goldstein commented on SSHD-1050: -- Found it - the {{ClientAuthUserService}} creates a new

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170599#comment-17170599 ] Lyor Goldstein commented on SSHD-1050: -- Actually this whole code is strange - the way I read, the

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170594#comment-17170594 ] Lyor Goldstein commented on SSHD-1050: -- {quote} What's the reason for the always false AuthFuture

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Thomas Wolf (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170592#comment-17170592 ] Thomas Wolf commented on SSHD-1050: --- Right. I shouldn't post spur-of-the-moment ideas :).  This needs

[jira] [Comment Edited] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170588#comment-17170588 ] Lyor Goldstein edited comment on SSHD-1050 at 8/4/20, 6:30 AM: --- {quote}

[jira] [Commented] (SSHD-1050) Race condition between early exceptions and AuthFuture

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170588#comment-17170588 ] Lyor Goldstein commented on SSHD-1050: -- {quote} Perhaps the connect future should be fulfilled only

[jira] [Commented] (SSHD-1049) comparison between SftpClient and sftp comand in Windows10

2020-08-04 Thread Lyor Goldstein (Jira)
[ https://issues.apache.org/jira/browse/SSHD-1049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17170584#comment-17170584 ] Lyor Goldstein commented on SSHD-1049: -- In a related issue - I was wondering how does SCP compare. I