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

Python code gives error HTTPProxyConnection.call() missing 2 required positional arguments: 'port' and 'timeout' #1090

Open
prashant1diwase opened this issue Aug 27, 2024 · 1 comment
Labels

Comments

@prashant1diwase
Copy link

prashant1diwase commented Aug 27, 2024

Describe the bug

When we tried to connect with the below details, it started giving the below error with pyvmomi 8.0.3.0.1

Error

HTTPProxyConnection.call() missing 2 required positional arguments: 'port' and 'timeout'

Reproduction steps

When we tried to connect with the below details, it started giving the below error with pyvmomi 8.0.3.0.1

Code

    server = "146.89.140.53"
    httpsProxy ='http://146.72.882.22:3128'
    proxy = urlparse(httpsProxy)

    smart_stub = SmartStubAdapter(
        host=server,
        httpProxyHost=proxy.hostname,
        httpProxyPort=proxy.port,
        disableSslCertValidation=True,
    )

Expected behavior

It should be able to connect

Additional context

Analysis

However, when we checked further there was a change in implementation for pyvmomi 8.0.3.0.1 compared with pyvmomi 8.0.2.0.1.
When we applied the fix given in #1085, by changing committed files [connect.py, Security.py, SoapAdapter.py] (9a8956f) it gave different error -

Error

[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1006)')

@DanielDraganov
Copy link
Collaborator

Hi @prashant1diwase
An additional fix was applied after 9a8956f because the prior introduced a bug in the proxy connection's host and port initialization.
Please checkout the latest pyVmomi from the master branch and try again.

I couldn't reproduce the SSL issue so if by some chance you encounter an SSL error please post the traceback.

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

No branches or pull requests

2 participants