[ https://issues.apache.org/jira/browse/DISPATCH-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17324009#comment-17324009 ]
ASF GitHub Bot commented on DISPATCH-1878: ------------------------------------------ ChugR commented on a change in pull request #1129: URL: https://github.com/apache/qpid-dispatch/pull/1129#discussion_r615043535 ########## File path: src/adaptors/tcp_adaptor.c ########## @@ -57,11 +57,15 @@ struct qdr_tcp_connection_t { qdr_delivery_t *outstream; bool ingress; bool flow_enabled; + bool incoming_started; bool egress_dispatcher; bool connector_closed;//only used if egress_dispatcher=true bool in_list; // This connection is in the adaptor's connections list - bool raw_closed_read; - bool raw_closed_write; + bool raw_closed_read; // proton event seen + bool raw_closed_write; // proton event seen or write_close called + bool raw_read_shutdown; // stream closed + bool read_eos_seen; + qd_buffer_list_t early_raw_read_bufs; // read from raw conn before ingress stream ready Review comment: This was on recommendation from proton developers: giving buffers to the read side is pretty harmless. Reading from raw connection before the ingress stream is ready is the first strategy of several that has worked. Over the wire the client has executed open-write-close before the TCP adaptor has even been notified that the connection has been accepted. Then follows a READ event which TCP can't handle since it has no stream or credit. Then follows a READ_CLOSED which still can't be handled since there is no stream or credit. Eventually the stream comes up and credit arrives but these events don't arrive in the proper context for reading the raw connection. One could schedule a wake but complicated state is required to signal that on a random wake it's time to read from a closed connection. -- 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 go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Client app not getting a response through tcpListener > ----------------------------------------------------- > > Key: DISPATCH-1878 > URL: https://issues.apache.org/jira/browse/DISPATCH-1878 > Project: Qpid Dispatch > Issue Type: Bug > Components: Protocol Adaptors > Affects Versions: 2.0.0 > Reporter: Fernando Giorgetti > Assignee: Charles E. Rolke > Priority: Major > Attachments: D-1878 @e148c.svg, > DISPATCH-1878_fail-pn-raw-closed-before-ingress-stream-set-up.svg, > dispatch-1878-trace.html > > > I have a tcp-echo server running locally through: > podman run -d --rm --name tcpecho -p 9090:9090 quay.io/skupper/tcp-go-echo > And I have a router configured with a tcpConnector to localhost at port 9090 > and a tcpListener at port 9999. > I am able to use nc (netcat) to send data to the tcp-echo directly (port > 9090) or through > the router (port 9999), if I run "nc 127.0.0.1 9999" and the send the data. > But if I run it as: "echo abcd | nc 127.0.0.1 9999" I am not seeing a > response, but if > I use port 9090 (original port) instead of 9999 (router tcpListener), then I > get the > response correctly. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org