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 CONNECT based proxies #799

Merged
merged 1 commit into from
Jul 23, 2019
Merged

support CONNECT based proxies #799

merged 1 commit into from
Jul 23, 2019

Commits on Jul 9, 2019

  1. fix pvmomi the proxy support

    - ensure the proxy configuration is passed to `__GetElementTree()`
    - reuse @blacksponge fix for the  `SSLTunnelConnection` class
    - add `test_ssl_tunnelwith_cert_file` test to validate that `cert_file`
      and `key_file` are passed as expected.
    
    The patch adds a new `HTTPProxyConnection` class to handle the `CONNECT`
    based proxies.
    The patch does not touch the `SSLTunnelConnection` class. Both are
    really difference, and I think it's a better approach.
    
    To summarize, if the user set `sslProxyPath`, we use the original method,
    if instead `httpProxyHost` is set, we use the new class. Both should not
    be used at the same time.
    
    closes: vmware#620
    closes: vmware#567
    closes: vmware#198
    goneri committed Jul 9, 2019
    Configuration menu
    Copy the full SHA
    52da80c View commit details
    Browse the repository at this point in the history