aminueza / terraform-provider-minio

Terraform provider for managing MinIO S3 buckets and IAM Users.
https://registry.terraform.io/providers/aminueza/minio
GNU Affero General Public License v3.0
243 stars 73 forks source link

fix: Allow for lowercase token in ldap dn #563

Closed fryshorts closed 6 months ago

fryshorts commented 6 months ago

Minio recently changed behaviour by using case sensitive ldap queries. This makes it necessary to specify the distinguished name for user/group policy attachments exactly in the same case as the server.

This merge request updates the validation regex to allow for lowercase LDIF field names in the dn (e.g. CN=foo vs cn=foo).

The relevant Minio MR: https://github.com/minio/minio/pull/19358

As an alternative the regex could be made case insensitive with (?i)^... but then also something like Cn=foo or cN=foo would be valid.