Skip to content

GH-626: Enable Streaming.Async for ChannelDirectTcpip#631

Merged
tomaswolf merged 1 commit into
apache:masterfrom
tomaswolf:gh-626
Nov 9, 2024
Merged

GH-626: Enable Streaming.Async for ChannelDirectTcpip#631
tomaswolf merged 1 commit into
apache:masterfrom
tomaswolf:gh-626

Conversation

@tomaswolf
Copy link
Copy Markdown
Member

ChannelDirectTcpip gives user code the ability to request a direct-tcpip forward explicitly, without listener on a local port. (Full local port forwarding with a listener on the local port is not implemented in ChannelDirectTcpip but in TcpipClientChannel.)

Complete the implementation to also support the asynchronous asyncIn and asyncOut streams. Factor out the stream setup from ChannelSession into a new intermediary class, and derive ChannelDirectTcpip from that new class. (Basing it on ChannelSession directly would be wrong; the channel would end up with the wrong type and also inherit the setEnv() method.)

Fixes #626.

ChannelDirectTcpip gives user code the ability to request a direct-tcpip
forward explicitly, without listener on a local port. (Full local port
forwarding with a listener on the local port is not implemented in
ChannelDirectTcpip but in TcpipClientChannel.)

Complete the implementation to also support the asynchronous asyncIn and
asyncOut streams. Factor out the stream setup from ChannelSession into a
new intermediary class, and derive ChannelDirectTcpip from that new
class. (Basing it on ChannelSession directly would be wrong; the channel
would end up with the wrong type and also inherit the setEnv() method.)
@tomaswolf tomaswolf merged commit d7a8d7d into apache:master Nov 9, 2024
@tomaswolf tomaswolf deleted the gh-626 branch April 25, 2025 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NPE in ChannelDirectTcpip when used with Streaming.Async

1 participant