ChangePasswordLdapDriver - Code Metrics - RainLoop/rainloop-webmail - Measure and Improve Code Quality continuously with Scrutinizer

ChangePasswordLdapDriver A
last analyzed 2019-04-17 21:09 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	240
Duplicated Lines	14.17 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

Metric	Value
dl	34
loc	240
rs	9.84
c	0
b	0
f	0
wmc	32
lcom	2
cbo	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	SetConfig()	0	10	1
A	SetAllowedEmails()	0	6	1
A	SetLogger()	0	9	2
A	PasswordChangePossibility()	0	5	3
F	ChangePassword()	34	118	23
A	getSalt()	0	13	2

How to fix Duplicated Code

<?php

class ChangePasswordLdapDriver implements \RainLoop\Providers\ChangePassword\ChangePasswordInterface
{
	/**
	 * @var string
	 */
	private $sHostName = '127.0.0.1';

	/**
	 * @var int
	 */
	private $iHostPort = 389;

	/**
	 * @var string
	 */
	private $sUserDnFormat = '';

	/**
	 * @var string
	 */
	private $sPasswordField = 'userPassword';

	/**
	 * @var string
	 */
	private $sPasswordEncType = 'SHA';

	/**
	 * @var \MailSo\Log\Logger
	 */
	private $oLogger = null;

	/**
	 * @var string
	 */
	private $sAllowedEmails = '';

	/**
	 * @param string $sHostName
	 * @param int $iHostPort
	 * @param string $sUserDnFormat
	 * @param string $sPasswordField
	 * @param string $sPasswordEncType
	 *
	 * @return \ChangePasswordLdapDriver
	 */
	public function SetConfig($sHostName, $iHostPort, $sUserDnFormat, $sPasswordField, $sPasswordEncType)
	{
		$this->sHostName = $sHostName;
		$this->iHostPort = $iHostPort;
		$this->sUserDnFormat = $sUserDnFormat;
		$this->sPasswordField = $sPasswordField;
		$this->sPasswordEncType = $sPasswordEncType;

		return $this;
	}

	/**
	 * @param string $sAllowedEmails
	 *
	 * @return \ChangePasswordLdapDriver
	 */
	public function SetAllowedEmails($sAllowedEmails)
	{
		$this->sAllowedEmails = $sAllowedEmails;

		return $this;
	}

	/**
	 * @param \MailSo\Log\Logger $oLogger
	 *
	 * @return \ChangePasswordLdapDriver
	 */
	public function SetLogger($oLogger)
	{
		if ($oLogger instanceof \MailSo\Log\Logger)
if ($x instanceof DoesNotExist) {
    // Do something.
}
		{
			$this->oLogger = $oLogger;
		}

		return $this;
	}

	/**
	 * @param \RainLoop\Account $oAccount
	 *
	 * @return bool
	 */
	public function PasswordChangePossibility($oAccount)
	{
		return $oAccount && $oAccount->Email() &&
			\RainLoop\Plugins\Helper::ValidateWildcardValues($oAccount->Email(), $this->sAllowedEmails);
	}

	/**
	 * @param \RainLoop\Model\Account $oAccount
	 * @param string $sPrevPassword
	 * @param string $sNewPassword
	 *
	 * @return bool
	 */
	public function ChangePassword(\RainLoop\Account $oAccount, $sPrevPassword, $sNewPassword)
	{
		$bResult = false;

		try
		{
			$sDomain = \MailSo\Base\Utils::GetDomainFromEmail($oAccount->Email());
			$sUserDn = \strtr($this->sUserDnFormat, array(
				'{domain}' => $sDomain,
				'{domain:dc}' => 'dc='.\strtr($sDomain, array('.' => ',dc=')),
				'{email}' => $oAccount->Email(),
				'{email:user}' => \MailSo\Base\Utils::GetAccountNameFromEmail($oAccount->Email()),
				'{email:domain}' => $sDomain,
				'{login}' => $oAccount->Login(),
				'{imap:login}' => $oAccount->Login(),
				'{imap:host}' => $oAccount->DomainIncHost(),
				'{imap:port}' => $oAccount->DomainIncPort(),
				'{gecos}' => function_exists('posix_getpwnam') ? posix_getpwnam($oAccount->Login()) : ''
			));

			$oCon = @\ldap_connect($this->sHostName, $this->iHostPort);
			if ($oCon)
			{
				if (!@\ldap_set_option($oCon, LDAP_OPT_PROTOCOL_VERSION, 3))
				{
					$this->oLogger->Write(
						'Failed to set LDAP Protocol version to 3, TLS not supported.',
						\MailSo\Log\Enumerations\Type::WARNING,
						'LDAP'
					);
				}
				else if (@!ldap_start_tls($oCon))
				{
					$this->oLogger->Write("ldap_start_tls failed: ".$oCon, \MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
				}

				if (!@\ldap_bind($oCon, $sUserDn, $sPrevPassword))

				{
					if ($this->oLogger)
					{
						$sError = $oCon ? @\ldap_error($oCon) : '';
						$iErrno = $oCon ? @\ldap_errno($oCon) : 0;

						$this->oLogger->Write('ldap_bind error: '.$sError.' ('.$iErrno.')',
							\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
					}

					return false;
				}
			}
			else
			{
				return false;
			}

			$sSshaSalt = '';
			$sShaPrefix = '{SHA}';
			$sEncodedNewPassword = $sNewPassword;
			switch (\strtolower($this->sPasswordEncType))
			{
				case 'ssha':
					$sSshaSalt = $this->getSalt(4);
					$sShaPrefix = '{SSHA}';
				case 'sha':
					switch (true)
					{
						default:
						case \function_exists('sha1'):

							$sEncodedNewPassword = $sShaPrefix.\base64_encode(\sha1($sNewPassword.$sSshaSalt, true).$sSshaSalt);
							break;
						case \function_exists('hash'):

							$sEncodedNewPassword = $sShaPrefix.\base64_encode(\hash('sha1', $sNewPassword, true).$sSshaSalt);
							break;
						case \function_exists('mhash') && defined('MHASH_SHA1'):
							$sEncodedNewPassword = $sShaPrefix.\base64_encode(\mhash(MHASH_SHA1, $sNewPassword).$sSshaSalt);
							break;
					}
					break;
				case 'md5':
					$sEncodedNewPassword = '{MD5}'.\base64_encode(\pack('H*', \md5($sNewPassword)));
					break;
				case 'crypt':
					$sEncodedNewPassword = '{CRYPT}'.\crypt($sNewPassword, $this->getSalt(2));
					break;
			}

			$aEntry = array();
			$aEntry[$this->sPasswordField] = (string) $sEncodedNewPassword;

			if (!!@\ldap_modify($oCon, $sUserDn, $aEntry))

			{
				$bResult = true;
			}
			else
			{
				if ($this->oLogger)
				{
					$sError = $oCon ? @\ldap_error($oCon) : '';
					$iErrno = $oCon ? @\ldap_errno($oCon) : 0;

					$this->oLogger->Write('ldap_modify error: '.$sError.' ('.$iErrno.')',
						\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
				}
			}
		}
		catch (\Exception $oException)
		{
			if ($this->oLogger)
			{
				$this->oLogger->WriteException($oException,
					\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
			}

			$bResult = false;
		}

		return $bResult;
	}

	/**
	 * @param int $iLength
	 *
	 * @return string
	 */
	private function getSalt($iLength)
	{
		$sChars = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789';
		$iCharsLength = \strlen($sChars);

		$sResult = '';
		while (\strlen($sResult) < $iLength)
		{
			$sResult .= \substr($sChars, \rand() % $iCharsLength, 1);
		}

		return $sResult;
	}
}


ChangePasswordLdapDriver A
last analyzed 2019-04-17 21:09 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

6 Methods

How to fix Duplicated Code

Duplicated Code

1. Missing dependencies

2. Missing use statement

1		<?php
2
3		class ChangePasswordLdapDriver implements \RainLoop\Providers\ChangePassword\ChangePasswordInterface
4		{
5		/**
6		* @var string
7		*/
8		private $sHostName = '127.0.0.1';
9
10		/**
11		* @var int
12		*/
13		private $iHostPort = 389;
14
15		/**
16		* @var string
17		*/
18		private $sUserDnFormat = '';
19
20		/**
21		* @var string
22		*/
23		private $sPasswordField = 'userPassword';
24
25		/**
26		* @var string
27		*/
28		private $sPasswordEncType = 'SHA';
29
30		/**
31		* @var \MailSo\Log\Logger
32		*/
33		private $oLogger = null;
34
35		/**
36		* @var string
37		*/
38		private $sAllowedEmails = '';
39
40		/**
41		* @param string $sHostName
42		* @param int $iHostPort
43		* @param string $sUserDnFormat
44		* @param string $sPasswordField
45		* @param string $sPasswordEncType
46		*
47		* @return \ChangePasswordLdapDriver
48		*/
49		public function SetConfig($sHostName, $iHostPort, $sUserDnFormat, $sPasswordField, $sPasswordEncType)
50		{
51		$this->sHostName = $sHostName;
52		$this->iHostPort = $iHostPort;
53		$this->sUserDnFormat = $sUserDnFormat;
54		$this->sPasswordField = $sPasswordField;
55		$this->sPasswordEncType = $sPasswordEncType;
56
57		return $this;
58		}
59
60		/**
61		* @param string $sAllowedEmails
62		*
63		* @return \ChangePasswordLdapDriver
64		*/
65		public function SetAllowedEmails($sAllowedEmails)
66		{
67		$this->sAllowedEmails = $sAllowedEmails;
68
69		return $this;
70		}
71
72		/**
73		* @param \MailSo\Log\Logger $oLogger
74		*
75		* @return \ChangePasswordLdapDriver
76		*/
77		public function SetLogger($oLogger)
78		{
79		if ($oLogger instanceof \MailSo\Log\Logger)
		0 ignored issues – show Bug introduced 2017-10-06 18:56 UTC by Report Bug Copy Issue Report The class `MailSo\Log\Logger` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
80		{
81		$this->oLogger = $oLogger;
82		}
83
84		return $this;
85		}
86
87		/**
88		* @param \RainLoop\Account $oAccount
89		*
90		* @return bool
91		*/
92		public function PasswordChangePossibility($oAccount)
93		{
94		return $oAccount && $oAccount->Email() &&
95		\RainLoop\Plugins\Helper::ValidateWildcardValues($oAccount->Email(), $this->sAllowedEmails);
96		}
97
98		/**
99		* @param \RainLoop\Model\Account $oAccount
100		* @param string $sPrevPassword
101		* @param string $sNewPassword
102		*
103		* @return bool
104		*/
105		public function ChangePassword(\RainLoop\Account $oAccount, $sPrevPassword, $sNewPassword)
106		{
107		$bResult = false;
108
109		try
110		{
111		$sDomain = \MailSo\Base\Utils::GetDomainFromEmail($oAccount->Email());
112		$sUserDn = \strtr($this->sUserDnFormat, array(
113		'{domain}' => $sDomain,
114		'{domain:dc}' => 'dc='.\strtr($sDomain, array('.' => ',dc=')),
115		'{email}' => $oAccount->Email(),
116		'{email:user}' => \MailSo\Base\Utils::GetAccountNameFromEmail($oAccount->Email()),
117		'{email:domain}' => $sDomain,
118		'{login}' => $oAccount->Login(),
119		'{imap:login}' => $oAccount->Login(),
120		'{imap:host}' => $oAccount->DomainIncHost(),
121		'{imap:port}' => $oAccount->DomainIncPort(),
122		'{gecos}' => function_exists('posix_getpwnam') ? posix_getpwnam($oAccount->Login()) : ''
123		));
124
125		$oCon = @\ldap_connect($this->sHostName, $this->iHostPort);
126		if ($oCon)
127		{
128		if (!@\ldap_set_option($oCon, LDAP_OPT_PROTOCOL_VERSION, 3))
129		{
130		$this->oLogger->Write(
131		'Failed to set LDAP Protocol version to 3, TLS not supported.',
132		\MailSo\Log\Enumerations\Type::WARNING,
133		'LDAP'
134		);
135		}
136		else if (@!ldap_start_tls($oCon))
137		{
138		$this->oLogger->Write("ldap_start_tls failed: ".$oCon, \MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
139		}
140
141	View Code Duplication	if (!@\ldap_bind($oCon, $sUserDn, $sPrevPassword))
		0 ignored issues – show Duplication introduced 2017-10-06 18:56 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
142		{
143		if ($this->oLogger)
144		{
145		$sError = $oCon ? @\ldap_error($oCon) : '';
146		$iErrno = $oCon ? @\ldap_errno($oCon) : 0;
147
148		$this->oLogger->Write('ldap_bind error: '.$sError.' ('.$iErrno.')',
149		\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
150		}
151
152		return false;
153		}
154		}
155		else
156		{
157		return false;
158		}
159
160		$sSshaSalt = '';
161		$sShaPrefix = '{SHA}';
162		$sEncodedNewPassword = $sNewPassword;
163		switch (\strtolower($this->sPasswordEncType))
164		{
165		case 'ssha':
166		$sSshaSalt = $this->getSalt(4);
167		$sShaPrefix = '{SSHA}';
168		case 'sha':
169		switch (true)
170		{
171		default:
172	View Code Duplication	case \function_exists('sha1'):
		0 ignored issues – show Duplication introduced 2017-10-06 18:56 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
173		$sEncodedNewPassword = $sShaPrefix.\base64_encode(\sha1($sNewPassword.$sSshaSalt, true).$sSshaSalt);
174		break;
175	View Code Duplication	case \function_exists('hash'):
		0 ignored issues – show Duplication introduced 2017-10-06 18:56 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
176		$sEncodedNewPassword = $sShaPrefix.\base64_encode(\hash('sha1', $sNewPassword, true).$sSshaSalt);
177		break;
178		case \function_exists('mhash') && defined('MHASH_SHA1'):
179		$sEncodedNewPassword = $sShaPrefix.\base64_encode(\mhash(MHASH_SHA1, $sNewPassword).$sSshaSalt);
180		break;
181		}
182		break;
183		case 'md5':
184		$sEncodedNewPassword = '{MD5}'.\base64_encode(\pack('H*', \md5($sNewPassword)));
185		break;
186		case 'crypt':
187		$sEncodedNewPassword = '{CRYPT}'.\crypt($sNewPassword, $this->getSalt(2));
188		break;
189		}
190
191		$aEntry = array();
192		$aEntry[$this->sPasswordField] = (string) $sEncodedNewPassword;
193
194	View Code Duplication	if (!!@\ldap_modify($oCon, $sUserDn, $aEntry))
		0 ignored issues – show Duplication introduced 2017-10-06 18:56 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
195		{
196		$bResult = true;
197		}
198		else
199		{
200		if ($this->oLogger)
201		{
202		$sError = $oCon ? @\ldap_error($oCon) : '';
203		$iErrno = $oCon ? @\ldap_errno($oCon) : 0;
204
205		$this->oLogger->Write('ldap_modify error: '.$sError.' ('.$iErrno.')',
206		\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
207		}
208		}
209		}
210		catch (\Exception $oException)
211		{
212		if ($this->oLogger)
213		{
214		$this->oLogger->WriteException($oException,
215		\MailSo\Log\Enumerations\Type::WARNING, 'LDAP');
216		}
217
218		$bResult = false;
219		}
220
221		return $bResult;
222		}
223
224		/**
225		* @param int $iLength
226		*
227		* @return string
228		*/
229		private function getSalt($iLength)
230		{
231		$sChars = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789';
232		$iCharsLength = \strlen($sChars);
233
234		$sResult = '';
235		while (\strlen($sResult) < $iLength)
236		{
237		$sResult .= \substr($sChars, \rand() % $iCharsLength, 1);
238		}
239
240		return $sResult;
241		}
242		}
243

RainLoop / rainloop-webmail

ChangePasswordLdapDriver A last analyzed 2019-04-17 21:09 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

6 Methods

How to fix Duplicated Code

Duplicated Code

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like

ChangePasswordLdapDriver A
last analyzed 2019-04-17 21:09 UTC