Tunneling Proxy for Non-WebSocket MQTT5 Connection

[elryno]

I have not been able to get a tunneling proxy connection to work using Mqtt5ClientBuilder. Looking through the Build() function shows that the HttpClientConnectionProxyOptions field m_proxyOptions is only used when m_websocketConfig.has_value() returns true. Is this intentional, or is this a bug?

It seems that this would fix the issue....

diff --git a/source/iot/Mqtt5Client.cpp b/source/iot/Mqtt5Client.cpp
index 46aaa8e..091ae0c 100644
--- a/source/iot/Mqtt5Client.cpp
+++ b/source/iot/Mqtt5Client.cpp
@@ -596,10 +596,11 @@ namespace Aws
                 {
                     m_options->WithHttpProxyOptions(m_websocketConfig->ProxyOptions.value());
                 }
-                else if (m_proxyOptions.has_value())
-                {
-                    m_options->WithHttpProxyOptions(m_proxyOptions.value());
-                }
+            }
+
+            if (m_proxyOptions.has_value())
+            {
+                m_options->WithHttpProxyOptions(m_proxyOptions.value());
             }
 
             return Crt::Mqtt5::Mqtt5Client::NewMqtt5Client(*m_options, m_allocator);

Can you confirm? If so, should I submit a PR?

[elryno]

Moving discussion to the awslabs/aws-crt-cpp repo.

[github-actions[bot]]

Hello! Reopening this discussion to make it searchable.