Update redux firewall rule name to allow underscores - resolves #2903

[RiverPhillips]

Update the regex validation to allow underscores in redux firewall rules, this is supported by azure

[tombuildsstuff]

LGTM - thanks for this @RiverPhillips

[ghostinthewires]

My update just simplifies the regex a little

[inkel]

Thank you so much! As a side note, the PR I was going to send also added a new test case for the (valid) _helloworld_ string.

[RiverPhillips]

LGTM - thanks for this @RiverPhillips

Thanks for another quick reviewal

[ghostinthewires]

@RiverPhillips - Just a heads up to say I amended your regex to \w in PR #2911

[RiverPhillips]

@ghostinthewires Much nicer, didn't realise that was possible

[ghostinthewires]

@RiverPhillips - No probs, theres loads of different options. You on Twitter?

[inkel]

Newer version looks definitely better. Thanks, @ghostinthewires !

[ghost]

This has been released in version 1.23.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 = "~> 1.23.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!