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

Add whitelist feature #303 #448

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

Conversation

dapicester
Copy link

@dapicester dapicester commented Nov 30, 2021

Allow a whitelist of addresses to access the network.
Pass a list of tuples (host, port) as whitelist argument.

whitelist = [
  ('localhost', 1234),
  ('127.0.0.1', 8080),
]
httpretty.enable(allow_net_connect=False, whitelist=whitelist)
# ...

Allow a whitelist of addresses to access the network.
Pass a list of tuples `(host, port)` as `whitelist` argument.
@dapicester dapicester mentioned this pull request Nov 30, 2021
@leikoilja
Copy link

@gabrielfalcao, can we merge this one?

@leikoilja
Copy link

@gabrielfalcao? :) i see you liked the commend above, but can we merge it? 😅

@t-pohl
Copy link

t-pohl commented Oct 29, 2023

@gabrielfalcao Can this be merged now? would really help me if this would be available.

@dapicester
Copy link
Author

Waiting for this to be merged, you can directly use my fork with pip or poetry.

@leikoilja
Copy link

Waiting for this to be merged, you can directly use my fork with pip or poetry.

@dapicester, which fork are you talking about?
https://github.com/dapicester/HTTPretty is 3 commits behind httpretty original source and does not have whitelisting functionality merged 🤔

@dapicester
Copy link
Author

@leikoilja you should use the feature/whitelist branch, not main.

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

Successfully merging this pull request may close these issues.

3 participants