Ldap::__construct() - Code Metrics - Inspection of "AD support" - simplesamlphp/simplesamlphp-module-ldap - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Pull Request — master (#32)

by Tim
created 2022-05-23 18:04 UTC
Ldap::__construct() A

↳ Parent: Ldap
Complexity

Conditions	2
Paths	2
Size

Total Lines	34
Code Lines	18
Duplication

Lines	0
Ratio	0 %
Importance

Changes	2
Bugs	1	Features	0
Metric	Value
cc	2
eloc	18
c	2
b	1
f	0
nc	2
nop	6
dl	0
loc	34
rs	9.6666
<?php

declare(strict_types=1);

namespace SimpleSAML\Module\ldap\Connector;

use SimpleSAML\Assert\Assert;
use SimpleSAML\Error;
use SimpleSAML\Logger;
use SimpleSAML\Module\ldap\ConnectorInterface;
use Symfony\Component\Ldap\Adapter\ExtLdap\Adapter;
use Symfony\Component\Ldap\Entry;
use Symfony\Component\Ldap\Exception\InvalidCredentialsException;
use Symfony\Component\Ldap\Ldap as LdapObject;

use function array_merge;
use function array_pop;
use function explode;
use function implode;
use function ini_get;
use function sprintf;
use function var_export;

class Ldap implements ConnectorInterface
{
    use LdapHelpers;

    /**
     * @var \Symfony\Component\Ldap\Adapter\ExtLdap\Adapter
     */
    protected Adapter $adapter;

    /**
     * @var \Symfony\Component\Ldap\Ldap
     */
    protected LdapObject $connection;


    /**
     * @param string $connection_strings
     * @param string $encryption
     * @param int $version
     * @param string $extension
     * @param bool $debug
     * @param array $options
     */
    public function __construct(
        string $connection_strings,
        string $encryption = 'ssl',
        int    $version = 3,
        string $extension = 'ext_ldap',
        /** @scrutinizer ignore-unused */ string $extension = 'ext_ldap',
        bool   $debug = false,
        array  $options = ['referrals' => false, 'network_timeout' => 3]
    ) {
        foreach (explode(' ', $connection_strings) as $connection_string) {
            Assert::regex($connection_string, '#^ldap[s]?:\/\/#');
        }

        Logger::debug(
            sprintf(
                "Setting up LDAP connection: host='%s', encryption=%s, version=%d, debug=%s, timeout=%d, referrals=%s.",
                $connection_strings,
                $encryption,
                $version,
                var_export($debug, true),
                $options['timeout'] ?? ini_get('default_socket_timeout'),
                var_export($options['referrals'] ?? false, true),
            ));

        $this->adapter = new Adapter(
            [
                'connection_string' => $connection_strings,
                'encryption'        => $encryption,
                'version'           => $version,
                'debug'             => $debug,
                'options'           => $options,
            ]
        );

        $this->connection = new LdapObject($this->adapter);
    }


    /**
     * @inheritDoc
     */
    public function bind(string $username, ?string $password): void
    {
        try {
            $this->connection->bind($username, strval($password));
        } catch (InvalidCredentialsException $e) {
            throw new Error\Error($this->resolveBindError($e));
        }

        Logger::debug(sprintf("LDAP bind(): Bind successful for DN '%s'.", $username));
    }


    /**
     * @inheritDoc
     */
    public function search(
        array  $searchBase,
        string $filter,
        array  $options,
        bool   $allowMissing
    ): ?Entry
    {
        $entry = null;

        foreach ($searchBase as $base) {
            $query  = $this->connection->query($base, $filter, $options);
            $result = $query->execute()->toArray();

            if (count($result) > 1) {
                throw new Error\Exception(
                    sprintf(
                        "LDAP search(): Found %d entries searching base '%s' for '%s'",
                        count($result),
                        $base,
                        $filter,
                    )
                );
            } elseif (count($result) === 1) {
                $entry = array_pop($result);
                $entry = array_pop(/** @scrutinizer ignore-type */ $result);
                break;
            } else {
                Logger::debug(
                    sprintf(
                        "LDAP search(): Found no entries searching base '%s' for '%s'",
                        $base,
                        $filter,
                    )
                );
            }
        }

        if ($entry === null && $allowMissing === false) {
            throw new Error\Exception(
                sprintf(
                    "Object not found using search base [%s] and filter '%s'",
                    implode(', ', $searchBase),
                    $filter
                )
            );
        }

        return $entry;
    }


    /**
     * @inheritDoc
     */
    public function searchForMultiple(
        array  $searchBase,
        string $filter,
        array  $options,
        bool   $allowMissing
    ): array
    {
        $results = [];

        foreach ($searchBase as $base) {
            $query   = $this->connection->query($base, $filter, $options);
            $result  = $query->execute()->toArray();
            $results = array_merge($results, $result);
            $results = array_merge($results, /** @scrutinizer ignore-type */ $result);

            Logger::debug(sprintf(
                "Library - LDAP search(): Found %d entries searching base '%s' for '%s'",
                count($result),
                $base,
                $filter,
            ));
        }

        if (empty($results) && ($allowMissing === false)) {
            throw new Error\Exception(
                sprintf(
                    "No Objects found using search base [%s] and filter '%s'",
                    implode(', ', $searchBase),
                    $filter
                )
            );
        }

        return $results;
    }


    /**
     * Resolve the message to a UI exception
     *
     * @param InvalidCredentialsException $e
     * @return string
     */
    protected function resolveBindError(InvalidCredentialsException $e): string
    protected function resolveBindError(/** @scrutinizer ignore-unused */ InvalidCredentialsException $e): string
    {
        return self::ERR_WRONG_PASS;
    }
}

simplesamlphp / simplesamlphp-module-ldap

Pull Request — master (#32)

Ldap::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
