Code cleanup.

Potherca · Potherca · commit 71e1fdcd714f · 2025-06-27T11:08:25.000+02:00
diff --git a/solid/lib/Controller/ServerController.php b/solid/lib/Controller/ServerController.php
@@ -239,11 +239,10 @@ public function authorize() {
 			}
 		}
 
-		// @CHECKME: Can more than one redirect_uri could be provided for custom schemes?
-		$parsedOrigin = parse_url($clientRegistration['redirect_uris'][0]);
+		$parsedOrigin = parse_url($redirectUri);
 		if (
-			$parsedOrigin['scheme'] != "https" &&
-			$parsedOrigin['scheme'] != "http" &&
+			$parsedOrigin['scheme'] !== "https" &&
+			$parsedOrigin['scheme'] !== "http" &&
 			!isset($_GET['customscheme'])
 		) {
 			$result = new JSONResponse('Custom schema');

Original file line number	Diff line number	Diff line change
`@@ -239,11 +239,10 @@ public function authorize() {`
`239`	`239`	`}`
`240`	`240`	`}`
`241`	`241`
`242`		`- // @CHECKME: Can more than one redirect_uri could be provided for custom schemes?`
`243`		`- $parsedOrigin = parse_url($clientRegistration['redirect_uris'][0]);`
	`242`	`+ $parsedOrigin = parse_url($redirectUri);`
`244`	`243`	`if (`
`245`		`- $parsedOrigin['scheme'] != "https" &&`
`246`		`- $parsedOrigin['scheme'] != "http" &&`
	`244`	`+ $parsedOrigin['scheme'] !== "https" &&`
	`245`	`+ $parsedOrigin['scheme'] !== "http" &&`
`247`	`246`	`!isset($_GET['customscheme'])`
`248`	`247`	`) {`
`249`	`248`	`$result = new JSONResponse('Custom schema');`