claranet / terraform-azurerm-storage-account

Terraform module for Azure Storage
Apache License 2.0
11 stars 17 forks source link

AzureRM 3.114.0 deprecated enable_https_traffic_only as superseded by https_traffic_only_enabled #17

Closed mloskot closed 2 months ago

mloskot commented 3 months ago

Community Note

Description

https://github.com/claranet/terraform-azurerm-storage-account/blob/506e7cb6851fc92e681adcd9b2463750d6c0ac5f/r-storage-account.tf#L20

triggers the deprecation warning:

│ Warning: Argument is deprecated
│...
│   on .terraform/modules/stack_genesis.log_analytics.storage_logs/r-storage-account.tf line 20, in resource "azurerm_storage_account" "storage":
│   20:   enable_https_traffic_only        = var.nfsv3_enabled ? false : var.https_traffic_only_enabled
│
│ The property `enable_https_traffic_only` has been superseded by `https_traffic_only_enabled` and will be removed in v4.0 of the AzureRM Provider.

I'm opening this issue as reminder to update this module - I will try to submit PR after summer holidays period.

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

azurerm_storage_account

Potential Terraform Configuration

No response

References

davetustin commented 2 months ago

I have also ran into this warning and was going to open an issue here to update it.

Shr3ps commented 2 months ago

Internal MR opened :), should be fixed/released this week

mloskot commented 2 months ago

Thank you!