Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support full URL and https schema for minio_server #58

Closed
binlab opened this issue Mar 4, 2020 · 5 comments
Closed

Add support full URL and https schema for minio_server #58

binlab opened this issue Mar 4, 2020 · 5 comments
Assignees

Comments

@binlab
Copy link

binlab commented Mar 4, 2020

Thanks for the provider and your work!
Found some problem with the connection to minio via https schema

provider "minio" {
  minio_server     = "https://minio.example.com"
  minio_region     = "us-east-1"
  minio_access_key = "*REDACTED*"
  minio_secret_key = "*REDACTED*"
}

and output:

$ terraform apply

Error: Endpoint url cannot have fully qualified paths.

  on main.tf line 1, in provider "minio":
   1: provider "minio" {

Can you look at this? Thanks!

@felladrin felladrin added the bug Something isn't working label Mar 4, 2020
@aminueza
Copy link
Owner

aminueza commented Mar 5, 2020

Hi @binlab,

Thank you for letting us know. I need to update the documentation for ssl enabled.
To run minio under https you have to set up minio_ssl = "true" in main.tf and remove the https:// from minio_server url. The new configuration should be like:

provider "minio" {
  minio_server     = "minio.example.com"
  minio_region     = "us-east-1"
  minio_access_key = "*REDACTED*"
  minio_secret_key = "*REDACTED*"
  minio_ssl = "true"
}

Please, could you test it? If you are gonna been through same issue, please let me know.

@felladrin felladrin removed the bug Something isn't working label Mar 5, 2020
@binlab
Copy link
Author

binlab commented Mar 6, 2020

Awesome, it works! Thanks, @aminueza!
And I would ask you please little improvements regarding TLS for a future if it is possible. Can you add parameters to configure CA certificates ca_cert_file (for self-signed certificates) and client - certificates, and key cert_file, key_file (for certificate-based authorization)? How it implemented in many Terraform providers for example in a Vault - https://www.terraform.io/docs/providers/vault/index.html
Maybe it needs a separate issue?

@aminueza
Copy link
Owner

aminueza commented Mar 6, 2020

I'm gonna open an issue @binlab.
Thank you for a great feedback. Please, if you find any great feature to add, please, let me know.
For now, I'll close this issue.

@aminueza aminueza closed this as completed Mar 6, 2020
@toabi
Copy link

toabi commented Mar 18, 2021

Documentation is still missing the minio_ssl option which fixes actual issues using this.

Because without the ssl option and an automatically redirecting ingress, it works sometimes (for bucket operations) but not for user/policy operations.

@andrii-sudak
Copy link

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants