Skip to content
This repository has been archived by the owner on Jan 12, 2021. It is now read-only.

Don't configure proxy from env twice #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Don't configure proxy from env twice #108

wants to merge 1 commit into from

Commits on Nov 26, 2019

  1. Don't configure proxy from env twice 

    The proxy env vars are already taken into consideration here https://github.com/terraform-providers/terraform-provider-mysql/blob/master/mysql/provider.go#L152

Setting them as default for `proxy` leads to the proxy being set as proxy itself and as forwarder (see https://github.com/terraform-providers/terraform-provider-mysql/blob/master/mysql/provider.go#L160).

This leads to the following error (with `all_proxy=socks5://127.0.0.1:1080`)
Could not connect to server: socks connect tcp localhost:1080->target_host:target_port: socks connect tcp localhost: 1080->localhost: 1080
    @pascal-hofmann
    pascal-hofmann committed Nov 26, 2019
    Configuration menu
    Copy the full SHA
    66adad4 View commit details
    Browse the repository at this point in the history