Metadata validation in azurerm_storage_container is more restrictive then azure portal

[daniel-anova]

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

Terraform Version

1.3.9

AzureRM Provider Version

3.35.0

Affected Resource(s)/Data Source(s)

azurerm_storage_container

Terraform Configuration Files

resource "azurerm_resource_group" "example" {
  name     = "example-resources"
  location = "West Europe"
}

resource "azurerm_storage_account" "example" {
  name                     = "examplestoraccount"
  resource_group_name      = azurerm_resource_group.example.name
  location                 = azurerm_resource_group.example.location
  account_tier             = "Standard"
  account_replication_type = "LRS"

  tags = {
    environment = "staging"
  }
}

resource "azurerm_storage_container" "example" {
  name                  = "container"
  storage_account_name  = azurerm_storage_account.example.name
  container_access_type = "private"

  metadata = {
    myKey = "value"
  }
}

Debug Output/Panic Output

N/A

Expected Behaviour

camelCase names should be allowed for key names.

Actual Behaviour

terraform enforces all lowercase as per changes done in #10723

Steps to Reproduce

1. terraform apply

Important Factoids

No response

References

Key case was enforced to lowercase by #10723, however in the Portal it's possible to create upper case labels.

[magodo]

The issue resides in the go-autorest:

terraform-provider-azurerm/vendor/github.com/Azure/go-autorest/autorest/preparer.go

Line 151 in 8c7510d

r.Header.Set(http.CanonicalHeaderKey(name), value)

[magodo]

@tombuildsstuff Do we have a plan to migrate the transport layer for giovanni, given go-autorest seems not going to fix: Azure/go-autorest#721

[tombuildsstuff]

@magodo see tombuildsstuff/giovanni#71, tombuildsstuff/giovanni#68 and tombuildsstuff/giovanni#75