Add support to disable public_network_access in azurerm_bot_service_azure_bot

[DennisLundtoft]

Is there an existing issue for this?

• I have searched the existing issues

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment and review the contribution guide to help.

Description

Add support to disable public_network_access in azurerm_bot_service_azure_bot
azurerm_bot_service_azure_bot seem to be missing the property to disable public network access. It seems to be supported in the SDK

New or Affected Resource(s)/Data Source(s)

azurerm_bot_service_azure_bot

Potential Terraform Configuration

resource "azurerm_bot_service_azure_bot" "example" {
  name                = "examplebot"
  resource_group_name = var.resource_group_name
  location            = var.location
  microsoft_app_id    = var.client_id
  sku                 = "F0"

  public_network_access_enabled = false
}

References

No response

[rcskosir]

@DennisLundtoft Thank you for taking the time to open this feature request!

[neil-yechenwei]

Thanks for raising this issue. I assume it has been supported by TF. See more details from https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/bot_channels_registration.html#public_network_access_enabled.

[jake-setford]

Thanks for raising this issue. I assume it has been supported by TF. See more details from https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/bot_channels_registration.html#public_network_access_enabled.

@neil-yechenwei thats for the old deprecated azurerm_bot_channels_registration @DennisLundtoft is talking about the latest offering azurerm_bot_service_azure_bot. I'm currently migrating a number of bots and have just hit this issue.

[drdamour]

looks like this was attempted to be added in #24125 but seems to be broken on a refresh...TF always determines the value is false for our bots

[rcskosir]

@drdamour Thank you for helping find the connecting PR. I am going to mark this issue as resolved with #24125 since the enhancement support was added with that PR. I have labeled your issue #24242 as a bug and work to fix it can be tracked via that issue. Thank you again for the help.

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.