Completed
Push — master ( 3fa81d...4ab573 )
by Christoph
26:17 queued 22s
created
apps/federation/lib/SyncJob.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 	}
25 25
 
26 26
 	protected function run($argument) {
27
-		$this->syncService->syncThemAll(function ($url, $ex): void {
27
+		$this->syncService->syncThemAll(function($url, $ex): void {
28 28
 			if ($ex instanceof \Exception) {
29 29
 				$this->logger->error("Error while syncing $url.", [
30 30
 					'exception' => $ex,
Please login to merge, or discard this patch.
apps/federation/lib/BackgroundJob/RequestSharedSecret.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 	 */
87 87
 	protected function run($argument) {
88 88
 		$target = $argument['url'];
89
-		$created = isset($argument['created']) ? (int)$argument['created'] : $this->time->getTime();
89
+		$created = isset($argument['created']) ? (int) $argument['created'] : $this->time->getTime();
90 90
 		$currentTime = $this->time->getTime();
91 91
 		$source = $this->urlGenerator->getAbsoluteURL('/');
92 92
 		$source = rtrim($source, '/');
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
 		$endPoint = $endPoints['shared-secret'] ?? $this->defaultEndPoint;
106 106
 
107 107
 		// make sure that we have a well formatted url
108
-		$url = rtrim($target, '/') . '/' . trim($endPoint, '/');
108
+		$url = rtrim($target, '/').'/'.trim($endPoint, '/');
109 109
 
110 110
 		try {
111 111
 			$result = $this->httpClient->post(
@@ -126,13 +126,13 @@  discard block
 block discarded – undo
126 126
 		} catch (ClientException $e) {
127 127
 			$status = $e->getCode();
128 128
 			if ($status === Http::STATUS_FORBIDDEN) {
129
-				$this->logger->info($target . ' refused to ask for a shared secret.');
129
+				$this->logger->info($target.' refused to ask for a shared secret.');
130 130
 			} else {
131
-				$this->logger->info($target . ' responded with a ' . $status . ' containing: ' . $e->getMessage());
131
+				$this->logger->info($target.' responded with a '.$status.' containing: '.$e->getMessage());
132 132
 			}
133 133
 		} catch (RequestException $e) {
134 134
 			$status = -1; // There is no status code if we could not connect
135
-			$this->logger->info('Could not connect to ' . $target);
135
+			$this->logger->info('Could not connect to '.$target);
136 136
 		} catch (\Throwable $e) {
137 137
 			$status = Http::STATUS_INTERNAL_SERVER_ERROR;
138 138
 			$this->logger->error($e->getMessage(), ['exception' => $e]);
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	 */
153 153
 	protected function reAddJob(array $argument): void {
154 154
 		$url = $argument['url'];
155
-		$created = isset($argument['created']) ? (int)$argument['created'] : $this->time->getTime();
155
+		$created = isset($argument['created']) ? (int) $argument['created'] : $this->time->getTime();
156 156
 		$token = $argument['token'];
157 157
 		$attempt = $this->getAttempt($argument) + 1;
158 158
 
Please login to merge, or discard this patch.
apps/federation/lib/BackgroundJob/GetSharedSecret.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 
74 74
 	protected function run($argument) {
75 75
 		$target = $argument['url'];
76
-		$created = isset($argument['created']) ? (int)$argument['created'] : $this->time->getTime();
76
+		$created = isset($argument['created']) ? (int) $argument['created'] : $this->time->getTime();
77 77
 		$currentTime = $this->time->getTime();
78 78
 		$source = $this->urlGenerator->getAbsoluteURL('/');
79 79
 		$source = rtrim($source, '/');
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 		$endPoint = $endPoints['shared-secret'] ?? $this->defaultEndPoint;
93 93
 
94 94
 		// make sure that we have a well formatted url
95
-		$url = rtrim($target, '/') . '/' . trim($endPoint, '/');
95
+		$url = rtrim($target, '/').'/'.trim($endPoint, '/');
96 96
 
97 97
 		$result = null;
98 98
 		try {
@@ -115,13 +115,13 @@  discard block
 block discarded – undo
115 115
 		} catch (ClientException $e) {
116 116
 			$status = $e->getCode();
117 117
 			if ($status === Http::STATUS_FORBIDDEN) {
118
-				$this->logger->info($target . ' refused to exchange a shared secret with you.');
118
+				$this->logger->info($target.' refused to exchange a shared secret with you.');
119 119
 			} else {
120
-				$this->logger->info($target . ' responded with a ' . $status . ' containing: ' . $e->getMessage());
120
+				$this->logger->info($target.' responded with a '.$status.' containing: '.$e->getMessage());
121 121
 			}
122 122
 		} catch (RequestException $e) {
123 123
 			$status = -1; // There is no status code if we could not connect
124
-			$this->logger->info('Could not connect to ' . $target, [
124
+			$this->logger->info('Could not connect to '.$target, [
125 125
 				'exception' => $e,
126 126
 			]);
127 127
 		} catch (\Throwable $e) {
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 				);
150 150
 			} else {
151 151
 				$this->logger->error(
152
-					'remote server "' . $target . '"" does not return a valid shared secret. Received data: ' . $body
152
+					'remote server "'.$target.'"" does not return a valid shared secret. Received data: '.$body
153 153
 				);
154 154
 				$this->trustedServers->setServerStatus($target, TrustedServers::STATUS_FAILURE);
155 155
 			}
Please login to merge, or discard this patch.
apps/federation/lib/Controller/OCSAuthAPIController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 	public function requestSharedSecret(string $url, string $token): DataResponse {
99 99
 		if ($this->trustedServers->isTrustedServer($url) === false) {
100 100
 			$this->throttler->registerAttempt('federationSharedSecret', $this->request->getRemoteAddress());
101
-			$this->logger->error('remote server not trusted (' . $url . ') while requesting shared secret');
101
+			$this->logger->error('remote server not trusted ('.$url.') while requesting shared secret');
102 102
 			throw new OCSForbiddenException();
103 103
 		}
104 104
 
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 		$localToken = $this->dbHandler->getToken($url);
108 108
 		if (strcmp($localToken, $token) > 0) {
109 109
 			$this->logger->info(
110
-				'remote server (' . $url . ') presented lower token. We will initiate the exchange of the shared secret.'
110
+				'remote server ('.$url.') presented lower token. We will initiate the exchange of the shared secret.'
111 111
 			);
112 112
 			throw new OCSForbiddenException();
113 113
 		}
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	public function getSharedSecret(string $url, string $token): DataResponse {
141 141
 		if ($this->trustedServers->isTrustedServer($url) === false) {
142 142
 			$this->throttler->registerAttempt('federationSharedSecret', $this->request->getRemoteAddress());
143
-			$this->logger->error('remote server not trusted (' . $url . ') while getting shared secret');
143
+			$this->logger->error('remote server not trusted ('.$url.') while getting shared secret');
144 144
 			throw new OCSForbiddenException();
145 145
 		}
146 146
 
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 			$this->throttler->registerAttempt('federationSharedSecret', $this->request->getRemoteAddress());
149 149
 			$expectedToken = $this->dbHandler->getToken($url);
150 150
 			$this->logger->error(
151
-				'remote server (' . $url . ') didn\'t send a valid token (got "' . $token . '" but expected "' . $expectedToken . '") while getting shared secret'
151
+				'remote server ('.$url.') didn\'t send a valid token (got "'.$token.'" but expected "'.$expectedToken.'") while getting shared secret'
152 152
 			);
153 153
 			throw new OCSForbiddenException();
154 154
 		}
Please login to merge, or discard this patch.