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

'wrapmodule' is pointless #9

Open
SeaHOH opened this issue Dec 2, 2019 · 0 comments
Open

'wrapmodule' is pointless #9

SeaHOH opened this issue Dec 2, 2019 · 0 comments

Comments

@SeaHOH
Copy link

SeaHOH commented Dec 2, 2019

def wrapmodule(module):
"""wrapmodule(module)
Attempts to replace a module's socket library with a SOCKS socket.
This will only work on modules that import socket directly into the
namespace; most of the Python Standard Library falls into this category.
"""
module.socket.socket = socksocket
module.socket.create_connection = sockcreateconn

This function is pointless, in fact it just made a monkey patch to socket.socket, cannot be applied to the given module only. Please change or remove it.

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

1 participant