terraform-provider-htpasswd icon indicating copy to clipboard operation
terraform-provider-htpasswd copied to clipboard

Exclude $ for salt value

Open loafoe opened this issue 1 year ago • 1 comments

See https://github.com/loafoe/terraform-provider-htpasswd/pull/74

loafoe avatar Apr 22 '24 13:04 loafoe

While adding a validation for this should be easy, your usage example generally relies on random_password resources. If you just puzzle these parts together, and throw an error from inside this provider, it might be intransparent why it failed, because no user actually created that salt consciously. If you just "expect" the random_password to be "correct", and then have the provider error, this might confusion for the actual user.

Those are just my few cents from a UX standpoint, I generally welcome the change in terms of safety :)

zoomoid avatar Apr 22 '24 14:04 zoomoid