@@ -28,54 +28,54 @@ |
||
28 | 28 | use OCP\Http\Client\IClientService; |
29 | 29 | |
30 | 30 | class RetryJob extends Job { |
31 | - /** @var IClientService */ |
|
32 | - private $clientService; |
|
33 | - /** @var IJobList */ |
|
34 | - private $jobList; |
|
35 | - /** @var string */ |
|
36 | - private $lookupServer = 'https://lookup.nextcloud.com/users'; |
|
31 | + /** @var IClientService */ |
|
32 | + private $clientService; |
|
33 | + /** @var IJobList */ |
|
34 | + private $jobList; |
|
35 | + /** @var string */ |
|
36 | + private $lookupServer = 'https://lookup.nextcloud.com/users'; |
|
37 | 37 | |
38 | - /** |
|
39 | - * @param IClientService|null $clientService |
|
40 | - * @param IJobList|null $jobList |
|
41 | - */ |
|
42 | - public function __construct(IClientService $clientService = null, |
|
43 | - IJobList $jobList = null) { |
|
44 | - if($clientService !== null) { |
|
45 | - $this->clientService = $clientService; |
|
46 | - } else { |
|
47 | - $this->clientService = \OC::$server->getHTTPClientService(); |
|
48 | - } |
|
49 | - if($jobList !== null) { |
|
50 | - $this->jobList = $jobList; |
|
51 | - } else { |
|
52 | - $this->jobList = \OC::$server->getJobList(); |
|
53 | - } |
|
54 | - } |
|
38 | + /** |
|
39 | + * @param IClientService|null $clientService |
|
40 | + * @param IJobList|null $jobList |
|
41 | + */ |
|
42 | + public function __construct(IClientService $clientService = null, |
|
43 | + IJobList $jobList = null) { |
|
44 | + if($clientService !== null) { |
|
45 | + $this->clientService = $clientService; |
|
46 | + } else { |
|
47 | + $this->clientService = \OC::$server->getHTTPClientService(); |
|
48 | + } |
|
49 | + if($jobList !== null) { |
|
50 | + $this->jobList = $jobList; |
|
51 | + } else { |
|
52 | + $this->jobList = \OC::$server->getJobList(); |
|
53 | + } |
|
54 | + } |
|
55 | 55 | |
56 | - protected function run($argument) { |
|
57 | - if($argument['retryNo'] === 5) { |
|
58 | - return; |
|
59 | - } |
|
56 | + protected function run($argument) { |
|
57 | + if($argument['retryNo'] === 5) { |
|
58 | + return; |
|
59 | + } |
|
60 | 60 | |
61 | - $client = $this->clientService->newClient(); |
|
61 | + $client = $this->clientService->newClient(); |
|
62 | 62 | |
63 | - try { |
|
64 | - $client->post($this->lookupServer, |
|
65 | - [ |
|
66 | - 'body' => json_encode($argument['dataArray']), |
|
67 | - 'timeout' => 10, |
|
68 | - 'connect_timeout' => 3, |
|
69 | - ] |
|
70 | - ); |
|
71 | - } catch (\Exception $e) { |
|
72 | - $this->jobList->add(RetryJob::class, |
|
73 | - [ |
|
74 | - 'dataArray' => $argument['dataArray'], |
|
75 | - 'retryNo' => $argument['retryNo'] + 1, |
|
76 | - ] |
|
77 | - ); |
|
63 | + try { |
|
64 | + $client->post($this->lookupServer, |
|
65 | + [ |
|
66 | + 'body' => json_encode($argument['dataArray']), |
|
67 | + 'timeout' => 10, |
|
68 | + 'connect_timeout' => 3, |
|
69 | + ] |
|
70 | + ); |
|
71 | + } catch (\Exception $e) { |
|
72 | + $this->jobList->add(RetryJob::class, |
|
73 | + [ |
|
74 | + 'dataArray' => $argument['dataArray'], |
|
75 | + 'retryNo' => $argument['retryNo'] + 1, |
|
76 | + ] |
|
77 | + ); |
|
78 | 78 | |
79 | - } |
|
80 | - } |
|
79 | + } |
|
80 | + } |
|
81 | 81 | } |
@@ -22,24 +22,24 @@ |
||
22 | 22 | $dispatcher = \OC::$server->getEventDispatcher(); |
23 | 23 | |
24 | 24 | $dispatcher->addListener('OC\AccountManager::userUpdated', function(\Symfony\Component\EventDispatcher\GenericEvent $event) { |
25 | - $user = $event->getSubject(); |
|
25 | + $user = $event->getSubject(); |
|
26 | 26 | |
27 | - $keyManager = new \OC\Security\IdentityProof\Manager( |
|
28 | - \OC::$server->getAppDataDir('identityproof'), |
|
29 | - \OC::$server->getCrypto() |
|
30 | - ); |
|
31 | - $updateLookupServer = new \OCA\LookupServerConnector\UpdateLookupServer( |
|
32 | - new \OC\Accounts\AccountManager(\OC::$server->getDatabaseConnection(), \OC::$server->getEventDispatcher()), |
|
33 | - \OC::$server->getConfig(), |
|
34 | - \OC::$server->getSecureRandom(), |
|
35 | - \OC::$server->getHTTPClientService(), |
|
36 | - $keyManager, |
|
37 | - new \OC\Security\IdentityProof\Signer( |
|
38 | - $keyManager, |
|
39 | - new \OC\AppFramework\Utility\TimeFactory(), |
|
40 | - \OC::$server->getUserManager() |
|
41 | - ), |
|
42 | - \OC::$server->getJobList() |
|
43 | - ); |
|
44 | - $updateLookupServer->userUpdated($user); |
|
27 | + $keyManager = new \OC\Security\IdentityProof\Manager( |
|
28 | + \OC::$server->getAppDataDir('identityproof'), |
|
29 | + \OC::$server->getCrypto() |
|
30 | + ); |
|
31 | + $updateLookupServer = new \OCA\LookupServerConnector\UpdateLookupServer( |
|
32 | + new \OC\Accounts\AccountManager(\OC::$server->getDatabaseConnection(), \OC::$server->getEventDispatcher()), |
|
33 | + \OC::$server->getConfig(), |
|
34 | + \OC::$server->getSecureRandom(), |
|
35 | + \OC::$server->getHTTPClientService(), |
|
36 | + $keyManager, |
|
37 | + new \OC\Security\IdentityProof\Signer( |
|
38 | + $keyManager, |
|
39 | + new \OC\AppFramework\Utility\TimeFactory(), |
|
40 | + \OC::$server->getUserManager() |
|
41 | + ), |
|
42 | + \OC::$server->getJobList() |
|
43 | + ); |
|
44 | + $updateLookupServer->userUpdated($user); |
|
45 | 45 | }); |
@@ -5,46 +5,46 @@ discard block |
||
5 | 5 | vendor_style('user_ldap', 'ui-multiselect/jquery.multiselect'); |
6 | 6 | |
7 | 7 | script('user_ldap', [ |
8 | - 'wizard/controller', |
|
9 | - 'wizard/configModel', |
|
10 | - 'wizard/view', |
|
11 | - 'wizard/wizardObject', |
|
12 | - 'wizard/wizardTabGeneric', |
|
13 | - 'wizard/wizardTabElementary', |
|
14 | - 'wizard/wizardTabAbstractFilter', |
|
15 | - 'wizard/wizardTabUserFilter', |
|
16 | - 'wizard/wizardTabLoginFilter', |
|
17 | - 'wizard/wizardTabGroupFilter', |
|
18 | - 'wizard/wizardTabAdvanced', |
|
19 | - 'wizard/wizardTabExpert', |
|
20 | - 'wizard/wizardDetectorQueue', |
|
21 | - 'wizard/wizardDetectorGeneric', |
|
22 | - 'wizard/wizardDetectorPort', |
|
23 | - 'wizard/wizardDetectorBaseDN', |
|
24 | - 'wizard/wizardDetectorFeatureAbstract', |
|
25 | - 'wizard/wizardDetectorUserObjectClasses', |
|
26 | - 'wizard/wizardDetectorGroupObjectClasses', |
|
27 | - 'wizard/wizardDetectorGroupsForUsers', |
|
28 | - 'wizard/wizardDetectorGroupsForGroups', |
|
29 | - 'wizard/wizardDetectorSimpleRequestAbstract', |
|
30 | - 'wizard/wizardDetectorFilterUser', |
|
31 | - 'wizard/wizardDetectorFilterLogin', |
|
32 | - 'wizard/wizardDetectorFilterGroup', |
|
33 | - 'wizard/wizardDetectorUserCount', |
|
34 | - 'wizard/wizardDetectorGroupCount', |
|
35 | - 'wizard/wizardDetectorEmailAttribute', |
|
36 | - 'wizard/wizardDetectorUserDisplayNameAttribute', |
|
37 | - 'wizard/wizardDetectorUserGroupAssociation', |
|
38 | - 'wizard/wizardDetectorAvailableAttributes', |
|
39 | - 'wizard/wizardDetectorTestAbstract', |
|
40 | - 'wizard/wizardDetectorTestLoginName', |
|
41 | - 'wizard/wizardDetectorTestBaseDN', |
|
42 | - 'wizard/wizardDetectorTestConfiguration', |
|
43 | - 'wizard/wizardDetectorClearUserMappings', |
|
44 | - 'wizard/wizardDetectorClearGroupMappings', |
|
45 | - 'wizard/wizardFilterOnType', |
|
46 | - 'wizard/wizardFilterOnTypeFactory', |
|
47 | - 'wizard/wizard' |
|
8 | + 'wizard/controller', |
|
9 | + 'wizard/configModel', |
|
10 | + 'wizard/view', |
|
11 | + 'wizard/wizardObject', |
|
12 | + 'wizard/wizardTabGeneric', |
|
13 | + 'wizard/wizardTabElementary', |
|
14 | + 'wizard/wizardTabAbstractFilter', |
|
15 | + 'wizard/wizardTabUserFilter', |
|
16 | + 'wizard/wizardTabLoginFilter', |
|
17 | + 'wizard/wizardTabGroupFilter', |
|
18 | + 'wizard/wizardTabAdvanced', |
|
19 | + 'wizard/wizardTabExpert', |
|
20 | + 'wizard/wizardDetectorQueue', |
|
21 | + 'wizard/wizardDetectorGeneric', |
|
22 | + 'wizard/wizardDetectorPort', |
|
23 | + 'wizard/wizardDetectorBaseDN', |
|
24 | + 'wizard/wizardDetectorFeatureAbstract', |
|
25 | + 'wizard/wizardDetectorUserObjectClasses', |
|
26 | + 'wizard/wizardDetectorGroupObjectClasses', |
|
27 | + 'wizard/wizardDetectorGroupsForUsers', |
|
28 | + 'wizard/wizardDetectorGroupsForGroups', |
|
29 | + 'wizard/wizardDetectorSimpleRequestAbstract', |
|
30 | + 'wizard/wizardDetectorFilterUser', |
|
31 | + 'wizard/wizardDetectorFilterLogin', |
|
32 | + 'wizard/wizardDetectorFilterGroup', |
|
33 | + 'wizard/wizardDetectorUserCount', |
|
34 | + 'wizard/wizardDetectorGroupCount', |
|
35 | + 'wizard/wizardDetectorEmailAttribute', |
|
36 | + 'wizard/wizardDetectorUserDisplayNameAttribute', |
|
37 | + 'wizard/wizardDetectorUserGroupAssociation', |
|
38 | + 'wizard/wizardDetectorAvailableAttributes', |
|
39 | + 'wizard/wizardDetectorTestAbstract', |
|
40 | + 'wizard/wizardDetectorTestLoginName', |
|
41 | + 'wizard/wizardDetectorTestBaseDN', |
|
42 | + 'wizard/wizardDetectorTestConfiguration', |
|
43 | + 'wizard/wizardDetectorClearUserMappings', |
|
44 | + 'wizard/wizardDetectorClearGroupMappings', |
|
45 | + 'wizard/wizardFilterOnType', |
|
46 | + 'wizard/wizardFilterOnTypeFactory', |
|
47 | + 'wizard/wizard' |
|
48 | 48 | ]); |
49 | 49 | |
50 | 50 | style('user_ldap', 'settings'); |
@@ -67,12 +67,12 @@ discard block |
||
67 | 67 | <li class="ldapSettingsTabs"><a href="#ldapSettings-1"><?php p($l->t('Advanced'));?></a></li> |
68 | 68 | </ul> |
69 | 69 | <?php if(OCP\App::isEnabled('user_webdavauth')) { |
70 | - print_unescaped('<p class="ldapwarning">'.$l->t('<b>Warning:</b> Apps user_ldap and user_webdavauth are incompatible. You may experience unexpected behavior. Please ask your system administrator to disable one of them.').'</p>'); |
|
71 | - } |
|
72 | - if(!function_exists('ldap_connect')) { |
|
73 | - print_unescaped('<p class="ldapwarning">'.$l->t('<b>Warning:</b> The PHP LDAP module is not installed, the backend will not work. Please ask your system administrator to install it.').'</p>'); |
|
74 | - } |
|
75 | - ?> |
|
70 | + print_unescaped('<p class="ldapwarning">'.$l->t('<b>Warning:</b> Apps user_ldap and user_webdavauth are incompatible. You may experience unexpected behavior. Please ask your system administrator to disable one of them.').'</p>'); |
|
71 | + } |
|
72 | + if(!function_exists('ldap_connect')) { |
|
73 | + print_unescaped('<p class="ldapwarning">'.$l->t('<b>Warning:</b> The PHP LDAP module is not installed, the backend will not work. Please ask your system administrator to install it.').'</p>'); |
|
74 | + } |
|
75 | + ?> |
|
76 | 76 | <?php require_once(__DIR__ . '/part.wizard-server.php'); ?> |
77 | 77 | <?php require_once(__DIR__ . '/part.wizard-userfilter.php'); ?> |
78 | 78 | <?php require_once(__DIR__ . '/part.wizard-loginfilter.php'); ?> |
@@ -9,19 +9,19 @@ |
||
9 | 9 | <p> |
10 | 10 | <select id="ldap_serverconfig_chooser" name="ldap_serverconfig_chooser"> |
11 | 11 | <?php if(count($_['serverConfigurationPrefixes']) === 0 ) { |
12 | - ?> |
|
12 | + ?> |
|
13 | 13 | <option value="" selected><?php p($l->t('1. Server'));?></option>'); |
14 | 14 | <?php |
15 | - } else { |
|
16 | - $i = 1; |
|
17 | - $sel = ' selected'; |
|
18 | - foreach($_['serverConfigurationPrefixes'] as $prefix) { |
|
19 | - ?> |
|
15 | + } else { |
|
16 | + $i = 1; |
|
17 | + $sel = ' selected'; |
|
18 | + foreach($_['serverConfigurationPrefixes'] as $prefix) { |
|
19 | + ?> |
|
20 | 20 | <option value="<?php p($prefix); ?>"<?php p($sel); $sel = ''; ?>><?php p($l->t('%s. Server:', array($i++)));?> <?php p(' '.$_['serverConfigurationHosts'][$prefix]); ?></option> |
21 | 21 | <?php |
22 | - } |
|
23 | - } |
|
24 | - ?> |
|
22 | + } |
|
23 | + } |
|
24 | + ?> |
|
25 | 25 | </select> |
26 | 26 | <button type="button" id="ldap_action_add_configuration" |
27 | 27 | name="ldap_action_add_configuration" class="icon-add icon-default-style" |
@@ -34,16 +34,16 @@ |
||
34 | 34 | $subject = (string)$_POST['ldap_clear_mapping']; |
35 | 35 | $mapping = null; |
36 | 36 | if($subject === 'user') { |
37 | - $mapping = new UserMapping(\OC::$server->getDatabaseConnection()); |
|
37 | + $mapping = new UserMapping(\OC::$server->getDatabaseConnection()); |
|
38 | 38 | } else if($subject === 'group') { |
39 | - $mapping = new GroupMapping(\OC::$server->getDatabaseConnection()); |
|
39 | + $mapping = new GroupMapping(\OC::$server->getDatabaseConnection()); |
|
40 | 40 | } |
41 | 41 | try { |
42 | - if(is_null($mapping) || !$mapping->clear()) { |
|
43 | - $l = \OC::$server->getL10N('user_ldap'); |
|
44 | - throw new \Exception($l->t('Failed to clear the mappings.')); |
|
45 | - } |
|
46 | - OCP\JSON::success(); |
|
42 | + if(is_null($mapping) || !$mapping->clear()) { |
|
43 | + $l = \OC::$server->getL10N('user_ldap'); |
|
44 | + throw new \Exception($l->t('Failed to clear the mappings.')); |
|
45 | + } |
|
46 | + OCP\JSON::success(); |
|
47 | 47 | } catch (\Exception $e) { |
48 | - OCP\JSON::error(array('message' => $e->getMessage())); |
|
48 | + OCP\JSON::error(array('message' => $e->getMessage())); |
|
49 | 49 | } |
@@ -37,24 +37,24 @@ discard block |
||
37 | 37 | |
38 | 38 | |
39 | 39 | try { |
40 | - $configurationOk = true; |
|
41 | - $conf = $connection->getConfiguration(); |
|
42 | - if ($conf['ldap_configuration_active'] === '0') { |
|
43 | - //needs to be true, otherwise it will also fail with an irritating message |
|
44 | - $conf['ldap_configuration_active'] = '1'; |
|
45 | - $configurationOk = $connection->setConfiguration($conf); |
|
46 | - } |
|
47 | - if ($configurationOk) { |
|
48 | - //Configuration is okay |
|
49 | - /* |
|
40 | + $configurationOk = true; |
|
41 | + $conf = $connection->getConfiguration(); |
|
42 | + if ($conf['ldap_configuration_active'] === '0') { |
|
43 | + //needs to be true, otherwise it will also fail with an irritating message |
|
44 | + $conf['ldap_configuration_active'] = '1'; |
|
45 | + $configurationOk = $connection->setConfiguration($conf); |
|
46 | + } |
|
47 | + if ($configurationOk) { |
|
48 | + //Configuration is okay |
|
49 | + /* |
|
50 | 50 | * Clossing the session since it won't be used from this point on. There might be a potential |
51 | 51 | * race condition if a second request is made: either this request or the other might not |
52 | 52 | * contact the LDAP backup server the first time when it should, but there shouldn't be any |
53 | 53 | * problem with that other than the extra connection. |
54 | 54 | */ |
55 | - \OC::$server->getSession()->close(); |
|
56 | - if ($connection->bind()) { |
|
57 | - /* |
|
55 | + \OC::$server->getSession()->close(); |
|
56 | + if ($connection->bind()) { |
|
57 | + /* |
|
58 | 58 | * This shiny if block is an ugly hack to find out whether anonymous |
59 | 59 | * bind is possible on AD or not. Because AD happily and constantly |
60 | 60 | * replies with success to any anonymous bind request, we need to |
@@ -63,24 +63,24 @@ discard block |
||
63 | 63 | * exception by the LDAP wrapper. We catch this. Other cases may |
64 | 64 | * pass (like e.g. expected syntax error). |
65 | 65 | */ |
66 | - try { |
|
67 | - $ldapWrapper->read($connection->getConnectionResource(), '', 'objectClass=*', array('dn')); |
|
68 | - } catch (\Exception $e) { |
|
69 | - if($e->getCode() === 1) { |
|
70 | - OCP\JSON::error(array('message' => $l->t('The configuration is invalid: anonymous bind is not allowed.'))); |
|
71 | - exit; |
|
72 | - } |
|
73 | - } |
|
74 | - OCP\JSON::success(array('message' |
|
75 | - => $l->t('The configuration is valid and the connection could be established!'))); |
|
76 | - } else { |
|
77 | - OCP\JSON::error(array('message' |
|
78 | - => $l->t('The configuration is valid, but the Bind failed. Please check the server settings and credentials.'))); |
|
79 | - } |
|
80 | - } else { |
|
81 | - OCP\JSON::error(array('message' |
|
82 | - => $l->t('The configuration is invalid. Please have a look at the logs for further details.'))); |
|
83 | - } |
|
66 | + try { |
|
67 | + $ldapWrapper->read($connection->getConnectionResource(), '', 'objectClass=*', array('dn')); |
|
68 | + } catch (\Exception $e) { |
|
69 | + if($e->getCode() === 1) { |
|
70 | + OCP\JSON::error(array('message' => $l->t('The configuration is invalid: anonymous bind is not allowed.'))); |
|
71 | + exit; |
|
72 | + } |
|
73 | + } |
|
74 | + OCP\JSON::success(array('message' |
|
75 | + => $l->t('The configuration is valid and the connection could be established!'))); |
|
76 | + } else { |
|
77 | + OCP\JSON::error(array('message' |
|
78 | + => $l->t('The configuration is valid, but the Bind failed. Please check the server settings and credentials.'))); |
|
79 | + } |
|
80 | + } else { |
|
81 | + OCP\JSON::error(array('message' |
|
82 | + => $l->t('The configuration is invalid. Please have a look at the logs for further details.'))); |
|
83 | + } |
|
84 | 84 | } catch (\Exception $e) { |
85 | - OCP\JSON::error(array('message' => $e->getMessage())); |
|
85 | + OCP\JSON::error(array('message' => $e->getMessage())); |
|
86 | 86 | } |
@@ -34,7 +34,7 @@ |
||
34 | 34 | $connection = new \OCA\User_LDAP\Connection($ldapWrapper, $prefix); |
35 | 35 | $configuration = $connection->getConfiguration(); |
36 | 36 | if (isset($configuration['ldap_agent_password']) && $configuration['ldap_agent_password'] !== '') { |
37 | - // hide password |
|
38 | - $configuration['ldap_agent_password'] = '**PASSWORD SET**'; |
|
37 | + // hide password |
|
38 | + $configuration['ldap_agent_password'] = '**PASSWORD SET**'; |
|
39 | 39 | } |
40 | 40 | OCP\JSON::success(array('configuration' => $configuration)); |
@@ -36,11 +36,11 @@ |
||
36 | 36 | // only legacy checkboxes (Advanced and Expert tab) need to be handled here, |
37 | 37 | // the Wizard-like tabs handle it on their own |
38 | 38 | $chkboxes = array('ldap_configuration_active', 'ldap_override_main_server', |
39 | - 'ldap_turn_off_cert_check'); |
|
39 | + 'ldap_turn_off_cert_check'); |
|
40 | 40 | foreach($chkboxes as $boxid) { |
41 | - if(!isset($_POST[$boxid])) { |
|
42 | - $_POST[$boxid] = 0; |
|
43 | - } |
|
41 | + if(!isset($_POST[$boxid])) { |
|
42 | + $_POST[$boxid] = 0; |
|
43 | + } |
|
44 | 44 | } |
45 | 45 | |
46 | 46 | $ldapWrapper = new OCA\User_LDAP\LDAP(); |
@@ -34,8 +34,8 @@ |
||
34 | 34 | $prefix = (string)$_POST['ldap_serverconfig_chooser']; |
35 | 35 | $helper = new \OCA\User_LDAP\Helper(\OC::$server->getConfig()); |
36 | 36 | if($helper->deleteServerConfiguration($prefix)) { |
37 | - OCP\JSON::success(); |
|
37 | + OCP\JSON::success(); |
|
38 | 38 | } else { |
39 | - $l = \OC::$server->getL10N('user_ldap'); |
|
40 | - OCP\JSON::error(array('message' => $l->t('Failed to delete the server configuration'))); |
|
39 | + $l = \OC::$server->getL10N('user_ldap'); |
|
40 | + OCP\JSON::error(array('message' => $l->t('Failed to delete the server configuration'))); |
|
41 | 41 | } |