Disable local authentication between Airlock Processor and Azure Service Bus in Function Binding #4277
Conversation
…ween airlock and servicebus queue
|
/help |
|
🤖 pr-bot 🤖 Hello! You can use the following commands: (in response to this comment from @guybartal) |
Unit Test Results0 tests 0 ✅ 0s ⏱️ Results for commit 7ad89d4. ♻️ This comment has been updated with latest results. |
guybartal
changed the title
|
/test |
|
🤖 pr-bot 🤖 🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/12854924727 (with refid (in response to this comment from @guybartal) |
| "SB_CONNECTION_STRING" = var.airlock_servicebus.default_primary_connection_string | ||
| "SB_CONNECTION_STRING__tenantId" = azurerm_user_assigned_identity.airlock_id.tenant_id | ||
| "SB_CONNECTION_STRING__clientId" = azurerm_user_assigned_identity.airlock_id.client_id | ||
| "SB_CONNECTION_STRING__credential" = "managedidentity" | ||
| "SB_CONNECTION_STRING__fullyQualifiedNamespace" = var.airlock_servicebus_fqdn |
There was a problem hiding this comment.
Consider doing this like I have in the EventGrid PR.
| "id": "Microsoft.Azure.Functions.ExtensionBundle", | ||
| "version": "[4.0.0, 5.0.0)" | ||
| "version": "[4.*, 5.0.0)" |
There was a problem hiding this comment.
So you have confirmed that the current versions we use don't support this?
| @@ -62,6 +62,9 @@ variable "airlock_servicebus" { | |||
| default_primary_connection_string = string | |||
There was a problem hiding this comment.
Have you tried to add the endpoint property here and checking if it will get populated automatically?
Resolves #4259
What is being addressed
Customers may want to disable local auth between azure functions and
service bus queues, this PR sets the requiered environment variables for Azure Function bindings in Airlock Processor module.
How is this addressed