Azure / azure-sdk-for-go

This repository is for active development of the Azure SDK for Go. For consumers of the SDK we recommend visiting our public developer docs at:
https://docs.microsoft.com/azure/developer/go/
MIT License
1.56k stars 801 forks source link

Service Bus: Custom Endpoint Address Support #17713

Open jsquire opened 2 years ago

jsquire commented 2 years ago

Summary

When using an Express Route circuit, there are some configurations which are not able to open the initial connection to the Service Bus service using the fully qualified namespace as the connection endpoint. In these cases, a custom endpoint address is needed to ensure the correct routing for the connection but should only be used when first establishing the connection; after that point, the fully qualified namespace is expected by the service for normal operations.

In order to support this scenario, an approach is needed to allow for the endpoint address for connections to be overridden with an explicit value, taking precedence over the default behavior of assuming the fully qualified namespace.

Scope of Work

Success Criteria

Related Issues and References

jsquire commented 2 years ago

//fyi: @richardpark-msft

richardpark-msft commented 1 year ago

Costing at 3 days, which I think should be sufficient.

We can configure the Hostname in the open frame, based on what we pass in the connection options, which is independent of the hostname we connect to via TCP. So this should just be code changes inside of azservicebus.

github-actions[bot] commented 2 months ago

Hi @jsquire, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

richardpark-msft commented 2 months ago

(this is a valid issue, it's just hasn't been done yet)