Merge pull request #2113 from nextcloud/backport/2076/stable32

sorbaugh · web-flow · commit a0a3ef00327e · 2025-09-10T11:27:19.000+02:00
[stable32] Fix federated teams with space-containing userid
diff --git a/lib/Tools/ActivityPub/NCSignature.php b/lib/Tools/ActivityPub/NCSignature.php
@@ -197,7 +197,7 @@ private function setIncomingClearSignature(NCSignedRequest $signedRequest): void
 			throw new SignatureException('missing elements in \'headers\'');
 		}
 
-		$target = strtolower($request->getMethod()) . ' ' . $request->getRequestUri();
+		$target = strtolower($request->getMethod()) . ' ' . urldecode($request->getRequestUri());
 		$estimated = ['(request-target): ' . $target];
 
 		foreach ($headers as $key) {

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ private function setIncomingClearSignature(NCSignedRequest $signedRequest): void`
`197`	`197`	`throw new SignatureException('missing elements in \'headers\'');`
`198`	`198`	`}`
`199`	`199`
`200`		`- $target = strtolower($request->getMethod()) . ' ' . $request->getRequestUri();`
	`200`	`+ $target = strtolower($request->getMethod()) . ' ' . urldecode($request->getRequestUri());`
`201`	`201`	`$estimated = ['(request-target): ' . $target];`
`202`	`202`
`203`	`203`	`foreach ($headers as $key) {`