Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

RFC: New module 'rtt' and a bunch of changes in aid of what it does #48

Merged
merged 5 commits into from
Apr 11, 2016
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Make all probe() functions accept and ignore arbitrary kwargs.
This is forward-compatibility for the next change, in which
a new kwarg (that none of them need, but a new module does)
will be added.
  • Loading branch information
zackw committed Mar 29, 2016
commit c41a346deee53c97cf6f5c46c88eaf9fdfa147c1
2 changes: 1 addition & 1 deletion src/modules/checktest.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ def fetch_page(exit_desc):
logger.debug("Exit relay %s passed the check test." % url)


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Probe the given exit relay and look for check.tp.o false negatives.
"""
Expand Down
2 changes: 1 addition & 1 deletion src/modules/cloudflared.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ def is_cloudflared(exit_fpr):
logger.info("Exit %s does not see a CAPTCHA." % exit_url)


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Check if exit relay sees a CloudFlare CAPTCHA.
"""
Expand Down
2 changes: 1 addition & 1 deletion src/modules/dnspoison.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ def resolve(exit_desc, domain, whitelist):
(domain, exit))


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Probe the given exit relay and check if all domains resolve as expected.
"""
Expand Down
2 changes: 1 addition & 1 deletion src/modules/dnssec.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ def test_dnssec(exit_fpr):
logger.critical("%s resolved domain to %s" % (exit_url, ipv4))


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Test if exit relay can resolve broken domain.
"""
Expand Down
2 changes: 1 addition & 1 deletion src/modules/patchingCheck.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ def run_check(exit_desc):
os.remove(tmp_file)


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Probe the given exit relay and look for modified binaries.
"""
Expand Down
2 changes: 1 addition & 1 deletion src/modules/testfds.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ def fetch_page(exit_desc):
logger.debug("Exit relay %s worked fine." % exit_url)


def probe(exit_desc, run_python_over_tor, run_cmd_over_tor):
def probe(exit_desc, run_python_over_tor, run_cmd_over_tor, **kwargs):
"""
Attempts to fetch a small web page and yells if this fails.
"""
Expand Down