Function _processPrintersPolicy has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.

Open

def _processPrintersPolicy(policyBasepath):
    logging.info("== Parsing a printer policy! ==")
    policyFile = "{}/User/Preferences/Printers/Printers.xml".format(policyBasepath)
    printerList = []
    # test

• Read up
  Read up

Function _processDrivesPolicy has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

def _processDrivesPolicy(policyBasepath):
    logging.info("== Parsing a drive policy! ==")
    policyFile = "{}/User/Preferences/Drives/Drives.xml".format(policyBasepath)
    shareList = []

• Read up
  Read up

Function searchOne has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.

Open

def searchOne(filter):
    """Searches the LDAP with a filter and returns the first found object

    :param filter: A valid ldap filter
    :type filter: str

• Read up
  Read up

File __init__.py has 270 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import binascii

class Krb5KeytabUtil:
    """
    A class to parse and create krb5keytab content.

Function _processFilters has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

def _processFilters(policies):
    filteredPolicies = []

    for policy in policies:
        if  not len(policy["filters"]) > 0:

• Read up
  Read up

Function patchKeytab has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

def patchKeytab():
    """
    Patches the `/etc/krb5.keytab` file. It inserts the correct hostname of the current computer.

    :return: True on success, False otherwise

• Read up
  Read up

Function printLogs has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

def printLogs(compact=False,anonymize=False):
    """
    Print logs of linuxmuster-linuxclient7 from `/var/log/syslog`.

    :param compact: If set to True, some stuff like time and date will be removed. Defaults to False

• Read up
  Read up

Function setup has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

def setup(domain=None, user=None):
    """
    Sets up the client to be able to act in a linuxmuster environment

    :param domain: The domain to join, defaults to the first discovered domain

• Read up
  Read up

Function unmountAllSharesOfUser has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

def unmountAllSharesOfUser(username):
    """
    Unmount all shares of a given user and safely delete the mountpoints and the parent directory.

    :param username: The username of the user

• Read up
  Read up

Function _clearUserHomes has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

def _clearUserHomes(unattended=False):
    print("\nCAUTION! This will delete all userhomes of AD users!")
    if not unattended and not _askStep("clear all user homes now", False):
        return True

• Read up
  Read up

Function _mountShare has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

def _mountShare(username, networkPath, shareName, hiddenShare, useCruidOfExecutingUser=False):

    mountpoint = _getShareMountpoint(networkPath, username, hiddenShare, shareName)

    mountCommandOptions = f"file_mode=0700,dir_mode=0700,sec=krb5,nodev,nosuid,mfsymlinks,nobrl,vers=3.0,user={username}"

• Read up
  Read up

Function _getRemoteHookScripts has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

def _getRemoteHookScripts(hookType):
    if not hookType in remoteScriptNames:
        return False, None

    rc, networkConfig = config.network()

• Read up
  Read up

Function prepareForImage has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

def prepareForImage(unattended=False):
    """Prepare the computer for creating an image

    :param unattended: If set to True, all questions will be answered with yes, defaults to False
    :type unattended: bool, optional

• Read up
  Read up

Function removeLinesInFileContainingString has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

def removeLinesInFileContainingString(filePath, forbiddenStrings):
    """
    Remove all lines containing a given string form a file.

    :param filePath: The path to the file

• Read up
  Read up

Function deleteAllInDirectory has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def deleteAllInDirectory(directory):
    """
    Delete all files in a given directory

    :param directory: The path of the directory

• Read up
  Read up

Function _stripComment has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def _stripComment(fileData):
    filedata_stripped = ''
    for line in fileData.split('\n'):
        if line[:1] == '#':
            continue

• Read up
  Read up

Function deleteFilesWithExtension has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def deleteFilesWithExtension(directory, extension):
    """
    Delete all files with a given extension in a given directory.

    :param directory: The path of the directory

• Read up
  Read up

Function _mountShare has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

def _mountShare(username, networkPath, shareName, hiddenShare, useCruidOfExecutingUser=False):

Function upgrade has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

def upgrade():
    """
    Performs an upgrade of the linuxmuster-linuxclient7. This is executed after the package is updated.

    :return: True on success, False otherwise

• Read up
  Read up

Function _unmountShare has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

def _unmountShare(mountpoint):
    # check if mountpoint exists
    if (not os.path.exists(mountpoint)) or (not os.path.isdir(mountpoint)):
        logging.warning(f"* Could not unmount {mountpoint}, it does not exist.")

• Read up
  Read up