Merge pull request #22215 from nextcloud/backport/22211/stable19

[stable19] remove logging message carrying no valuable information
This commit is contained in:
Morris Jobke 2020-08-12 13:50:40 +02:00 committed by GitHub
commit 527bec1706
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 8 deletions

View File

@ -46,6 +46,7 @@
namespace OCA\User_LDAP; namespace OCA\User_LDAP;
use DomainException;
use OC\HintException; use OC\HintException;
use OC\Hooks\PublicEmitter; use OC\Hooks\PublicEmitter;
use OC\ServerNotAvailableException; use OC\ServerNotAvailableException;
@ -1518,11 +1519,11 @@ class Access extends LDAPUtility {
* @param string[] $searchAttributes needs to have at least two attributes, * @param string[] $searchAttributes needs to have at least two attributes,
* otherwise it does not make sense :) * otherwise it does not make sense :)
* @return string the final filter part to use in LDAP searches * @return string the final filter part to use in LDAP searches
* @throws \Exception * @throws DomainException
*/ */
private function getAdvancedFilterPartForSearch($search, $searchAttributes) { private function getAdvancedFilterPartForSearch($search, $searchAttributes) {
if (!is_array($searchAttributes) || count($searchAttributes) < 2) { if (!is_array($searchAttributes) || count($searchAttributes) < 2) {
throw new \Exception('searchAttributes must be an array with at least two string'); throw new DomainException('searchAttributes must be an array with at least two string');
} }
$searchWords = explode(' ', trim($search)); $searchWords = explode(' ', trim($search));
$wordFilters = []; $wordFilters = [];
@ -1553,12 +1554,8 @@ class Access extends LDAPUtility {
if ($haveMultiSearchAttributes && strpos(trim($search), ' ') !== false) { if ($haveMultiSearchAttributes && strpos(trim($search), ' ') !== false) {
try { try {
return $this->getAdvancedFilterPartForSearch($search, $searchAttributes); return $this->getAdvancedFilterPartForSearch($search, $searchAttributes);
} catch (\Exception $e) { } catch (DomainException $e) {
\OCP\Util::writeLog( // Creating advanced filter for search failed, falling back to simple method. Edge case, but valid.
'user_ldap',
'Creating advanced filter for search failed, falling back to simple method.',
ILogger::INFO
);
} }
} }