[ https://issues.apache.org/jira/browse/SSHD-964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17022311#comment-17022311 ]
Lyor Goldstein edited comment on SSHD-964 at 1/23/20 5:52 PM: -------------------------------------------------------------- {quote}I will try your suggested code changes into AbstractChannel class in methods sendEof() and writePacket() {quote} Thanks [~roberto.deandrea]. On that note, I am having 2nd thoughts about this piece of code: {code:java|title=AbstractChannel#sendEof} if (eofReceived.get()) { return; } {code} I am not sure why this is correct... I have re-read the relevant section in RFC4254 and I cannot really find an interpretation that mandates this behavior. What was the rationale behind it when you made your changes ? {quote}NOT REMOVING shutdownOutputStream() method in TcpipClientChannel class IS NOT ACCEPTABLE {quote} Like I said, I am not against removing it - but I need some more time to weigh the consequences... I was able to trace its introduction to SSHD-902 - I would appreciate it if you could also take a look at it and give me your opinion on it. was (Author: lgoldstein): {quote} I will try your suggested code changes into AbstractChannel class in methods sendEof() and writePacket() {quote} Thanks [~roberto.deandrea]. On that note, I am having 2nd thoughts about this piece of code: {code:java|title=AbstractChannel#sendEof} if (eofReceived.get()) { return; } {code} I am not sure why this is correct... I have re-read the relevant section in RFC4254 and I cannot really find an interpretation that mandates this behavior. What was the rationale behind it when you made your changes ? {quote} NOT REMOVING shutdownOutputStream() method in TcpipClientChannel class IS NOT ACCEPTABLE {quote} Like I said, I am not against removing it - but I need some more time to weigh the consequences... > SSH_MSG_CHANNEL_EOF never sent or received for local and remote port > forwarding > ------------------------------------------------------------------------------- > > Key: SSHD-964 > URL: https://issues.apache.org/jira/browse/SSHD-964 > Project: MINA SSHD > Issue Type: Bug > Affects Versions: 2.3.0, 2.3.1 > Reporter: Roberto Deandrea > Assignee: Lyor Goldstein > Priority: Major > > We found that the message SSH_MSG_CHANNEL_EOF is never sent or received for > local and remote port forwarding connections. > We found probably the issue and we changed the code locally in a couple of > java classes to fix this issue. > Let me know if we can commit our changes in a pull request (into a specific > Apache SSHD branch) so that you can peer review the changes and decide if > you can accept them... > > Kind Regards > Roberto Deandrea > -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org For additional commands, e-mail: dev-h...@mina.apache.org