Update name validation for azurerm_storage_share_directory

[neil-yechenwei]

fixes #8341
fixes #8168

--- PASS: TestAccAzureRMStorageShareDirectory_basic (210.22s)
--- PASS: TestAccAzureRMStorageShareDirectory_uppercase (211.34s)
--- PASS: TestAccAzureRMStorageShareDirectory_complete (220.86s)
--- PASS: TestAccAzureRMStorageShareDirectory_requiresImport (227.83s)
--- PASS: TestAccAzureRMStorageShareDirectory_update (269.48s)
--- PASS: TestAccAzureRMStorageShareDirectory_nested (317.45s)

[manicminer]

@neil-yechenwei Thanks for this update. Hope you don't mind, I've pushed some extra changes to support single character directory names, and ensure each directory component conforms. Also, in my testing, double dashes in filenames seemed to work so I removed that check.

[WodansSon]

@neil-yechenwei Thanks for the PR! It mostly LGTM, however I did leave one very minor comment. Thanks again for the PR 🚀

[neil-yechenwei]

@manicminer , thanks for your commit, after tested, but I found the name can start with number and hyphen. So I updated PR by newest name validation rules.

[neil-yechenwei]

@WodansSon , Thanks for your comment. I've updated code according by the newest name validation rules. Please help to have a look. Thanks.

Changes were made

[WodansSon]

@neil-yechenwei Thanks for pushing those updates, LGTM! 🚀

[WodansSon]

[ghost]

This has been released in version 2.27.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.27.0"
}
# ... other configuration ...

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!