basundhara-c

Thanks for your reply @alyssawilk ! I have described the current design in some detail as we have some crucial changes in envoy core in order to make this work....

Hi @alyssawilk @wbpcode @mattklein123! I just wanted to follow up on the above design proposal detailing where we had to make envoy core changes to get reverse connections to work!...

@alyssawilk thanks for the suggestion! We were looking into this and it might be possible, we'll be trying it out internally and reaching out when we have changes! @agrawroh I'm...

@alyssawilk, on digging deeper, we find that the http connection pool's [base implementation](https://github.com/envoyproxy/envoy/blob/e8da3a53d9d5d68c907413442416cc20f3f4b9ce/source/common/http/conn_pool_base.h#L49) calls the dispatcher (via Host) to create a [CreateConnectionData](https://github.com/envoyproxy/envoy/blob/e8da3a53d9d5d68c907413442416cc20f3f4b9ce/source/common/http/conn_pool_base.h#L121) object that wraps over the connection object and...

That should work, thank you!

Thanks a lot @alyssawaik, we attended an Envoy meeting wherein Greg suggested something on the same lines. I am currently exploring this option!

@botengyao thanks a lot for the suggestion on obtaining the slot from the context! I am trying that out along with an attempt to move the code in [extensions](https://github.com/envoyproxy/envoy/pull/37367) to...

@botengyao we have added a bunch of changes according to your suggestion, namely: - Removed the getters and setters for connection handler and cluster manager - The RCManager and RCHandler...

@botengyao I wanted to discuss the connection close sequence. The main issue is on the initiator side where [we pass the socket to the listener](https://github.com/nutanix/envoyproxy-envoy/blob/ae22e8a8f5d779146bdadcf5805fdc2d9be03b12/contrib/reverse_connection/bootstrap/source/reverse_connection_initiator.cc#L309) which will henceforth own it...

@botengyao I added the following in the last commit: - Added a flag "reverse_conn_force_local_reply_" to send a local reply for reverse connections instead of the runtime guard - Changed the...