Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Azure Managed identities #368

Open
boblangley opened this issue Aug 30, 2023 · 9 comments
Open

Add support for Azure Managed identities #368

boblangley opened this issue Aug 30, 2023 · 9 comments
Labels

Comments

@boblangley
Copy link
Member

boblangley commented Aug 30, 2023

Describe the feature.

Is your feature related to a problem? Please describe.

Currently there is no way to configure the transport for managed identity, instead the package assumes a connection string will be used.

Describe the requested feature

Provide necessary support to the package to support managed identity connection configuration for the Azure Service Bus transport, ideally integrating with existing norms for connection configuration when using managed identity.

Describe alternatives you've considered

None

Additional Context

The missing feature was raised via:

@andreasohlund
Copy link
Member

I think the workaround mentioned in Particular/NServiceBus.AzureFunctions.InProcess.ServiceBus#549 would work here as well until this gets addressed

@DavidBoike
Copy link
Member

@andreasohlund I don't believe that workaround will work.

The Azure Service Bus transport public API only allows setting the token credential through the constructor or via a TransportExtensions extension method. The Worker model owns the usage of the constructor and as shown in this issue using configuration.AdvancedConfiguration.UseTransport(…) with a properly configured transport doesn't appear to work.

@Jokelab
Copy link

Jokelab commented Oct 21, 2023

Subscribed to this one, since we're really waiting for this feature in Azure Functions (isolated).

@mennolaan
Copy link

Also subscribing to this, much needed.

@mennolaan
Copy link

Btw there was an attempt for a pr: Particular/NServiceBus.AzureFunctions.InProcess.ServiceBus#718

It seems like the fix is fairly simple. Is there a reason why managed identity is not supported by particular?

It seems like the only way is to fork the repo an build a custom artifact and use that, but that is just silly.

@dvdstelt
Copy link
Member

The PR was exactly that, an attempt that was unfortunately abandoned because of other priorities. I also don't have more details about that PR than that.
We never really make promises about when we pick up or release features like this, but I can tell you that managed identity is very high on our radar for possibly Q1 2024.

@sanfordn
Copy link

We've been able to use Managed Identity with NServiceBus.AzureFunctions.Worker.ServiceBus version 4.0.0-rc.2. Not sure how exactly it all works, but I thought it might be worth mentioning here.

@mennolaan
Copy link

Hi Guys,

Any news on this subject?

Cheers.

@WilliamBZA
Copy link
Member

Yes! It's actually queued up to be worked on soon! There will be movement on this coming soon

@ramonsmits ramonsmits changed the title Add support for managed identity Add support for Azure Managed identities Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants