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

Support https endpoint #609

Closed
ghost opened this issue Dec 31, 2018 · 7 comments
Closed

Support https endpoint #609

ghost opened this issue Dec 31, 2018 · 7 comments

Comments

@ghost
Copy link

ghost commented Dec 31, 2018

Hello,

Please add support for https connection in provider configuration. Currently, all requests to AliCloud API are sent via HTTP protocol which is not secure and contains sensitive data like access key.

Example:

provider "alicloud" {
   access_key = "..."
   secret_key = "..."
   region = "...region..."
   https = true
}

This issue is very urgent from a security perspective.

Thanks,

@ozbillwang
Copy link

ozbillwang commented Dec 31, 2018

I am curious. May I know how you get the endpoint url? How do you know it is http, not https?

When I enable debug (TF_LOG=DEBUG terraform apply) I can see some extra debug logs, but these logs are not that much.

@ghost
Copy link
Author

ghost commented Dec 31, 2018

I did Wireshark inspections to see the requests designated to AliCloud endpoints and found that they are http based.

I would be a nice feature in terraform to include request logs if TF_LOG debug applied.

@chanind
Copy link
Contributor

chanind commented Jan 3, 2019

Wow that's scary! I tried to fix this in a PR here: #613. Does this fix the https issues you saw?

@xiaozhu36
Copy link
Member

xiaozhu36 commented Jan 9, 2019

HI @ninja-at-work Currently, all of services have supported https and please update to release 1.27.0.

@chanind
Copy link
Contributor

chanind commented Jan 9, 2019

It looks like OTS is still using HTTP fd70bf9. Maybe we should update the docs with a warning that using OTS resources is insecure?

@xiaozhu36
Copy link
Member

Hi @chanind I am sorry. I am working on resolving ots supports https and it will support in the next version.

@chanind
Copy link
Contributor

chanind commented Jan 10, 2019

Ok thanks for your hard work @xiaozhu36!

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

3 participants