search

Azure / azure-sdk-for-python

This repository is for active development of the Azure SDK for Python. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/python/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-python.
MIT License
4.58k stars 2.79k forks source link

Proxy coonection #35659

Closed jay-iyengar closed 4 months ago

jay-iyengar commented 5 months ago

Hi,

When I tried connect to one of the AMQP with the proxy it doesn't work. When I checked wire shark it's not really connecting via the proxy. It would be great if this could be fixed. I get below error and its connecting directly without proxy

params = {"http_proxy": {"proxy_hostname": "http://127.0.0.1", "proxy_port": 9000}} ServiceBusClient("ServiceBusNamespace", token_credential, **params)

Please confirm target hostname exists: b'elexon-iris.servicebus.windows.net'

Thanks Jay

github-actions[bot] commented 5 months ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @EldertGrootenboer.

kashifkhan commented 4 months ago

Thank you for the feedback @jay-iyengar . We will investigate and get back to you asap.

l0lawrence commented 4 months ago

Hi @jay-iyengar, could you share what version of the package you are using, as well as logs if you have them? Was this working for you previously?

github-actions[bot] commented 4 months ago

Hi @jay-iyengar. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

jay-iyengar commented 4 months ago

Hi, Apologies, I was using the this version azure-servicebus==7.8.0, just now upgraded to azure-servicebus==7.12.2 and thats seems to be working fine. Thanks for looking into it. Kind Regards Jay

kashifkhan commented 4 months ago

Thank you for confirming @jay-iyengar