mirror of
https://github.com/certbot/certbot.git
synced 2026-07-26 07:39:52 +02:00
Adrien FerrandandBrad Warren
56f609d4f5
Fixes #6850 This PR makes the last corrections needed to run all unit tests on Windows: add a function to check if a hook is executable in a cross-platform compatible way handle correctly the PATH surgery for Windows during hook execution handle correctly an account compatibility over both ACMEv1 and ACMEv2 remove (finally!) the @broken_on_windows decorator. * Fix account_tests * Fix hook executable test * Remove the temporary decorator @broken_on_windows * Fix util_test * No broken unit test on Windows anymore * More elegant mock * Fix context manager * Adapt coverage * Corrections * Adapt coverage * Forbid os.access
59 lines
1.7 KiB
Python
59 lines
1.7 KiB
Python
"""Plugin utilities."""
|
|
import logging
|
|
|
|
from certbot import util
|
|
from certbot.compat import os
|
|
from certbot.compat.misc import STANDARD_BINARY_DIRS
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
|
|
def get_prefixes(path):
|
|
"""Retrieves all possible path prefixes of a path, in descending order
|
|
of length. For instance,
|
|
(linux) /a/b/c returns ['/a/b/c', '/a/b', '/a', '/']
|
|
(windows) C:\\a\\b\\c returns ['C:\\a\\b\\c', 'C:\\a\\b', 'C:\\a', 'C:']
|
|
:param str path: the path to break into prefixes
|
|
|
|
:returns: all possible path prefixes of given path in descending order
|
|
:rtype: `list` of `str`
|
|
"""
|
|
prefix = os.path.normpath(path)
|
|
prefixes = []
|
|
while prefix:
|
|
prefixes.append(prefix)
|
|
prefix, _ = os.path.split(prefix)
|
|
# break once we hit the root path
|
|
if prefix == prefixes[-1]:
|
|
break
|
|
return prefixes
|
|
|
|
|
|
def path_surgery(cmd):
|
|
"""Attempt to perform PATH surgery to find cmd
|
|
|
|
Mitigates https://github.com/certbot/certbot/issues/1833
|
|
|
|
:param str cmd: the command that is being searched for in the PATH
|
|
|
|
:returns: True if the operation succeeded, False otherwise
|
|
"""
|
|
path = os.environ["PATH"]
|
|
added = []
|
|
for d in STANDARD_BINARY_DIRS:
|
|
if d not in path:
|
|
path += os.pathsep + d
|
|
added.append(d)
|
|
|
|
if any(added):
|
|
logger.debug("Can't find %s, attempting PATH mitigation by adding %s",
|
|
cmd, os.pathsep.join(added))
|
|
os.environ["PATH"] = path
|
|
|
|
if util.exe_exists(cmd):
|
|
return True
|
|
expanded = " expanded" if any(added) else ""
|
|
logger.debug("Failed to find executable %s in%s PATH: %s", cmd,
|
|
expanded, path)
|
|
return False
|