Merge pull request #769 from nextcloud/fix/noid/redirect-hard-idp

juliusknorr · web-flow · commit 64ec7c982488 · 2023-10-09T12:20:41.000+02:00
fix(login): do not hardcode IdP to 1 on redirect
diff --git a/appinfo/app.php b/appinfo/app.php
@@ -199,7 +199,7 @@
 		[
 			'requesttoken' => $csrfToken->getEncryptedValue(),
 			'originalUrl' => $originalUrl,
-			'idp' => 1,
+			'idp' => array_keys($configuredIdps)[0] ?? '',
 		]
 	);
 	header('Location: '.$targetUrl);

Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,7 @@`
`199`	`199`	`[`
`200`	`200`	`'requesttoken' => $csrfToken->getEncryptedValue(),`
`201`	`201`	`'originalUrl' => $originalUrl,`
`202`		`- 'idp' => 1,`
	`202`	`+ 'idp' => array_keys($configuredIdps)[0] ?? '',`
`203`	`203`	`]`
`204`	`204`	`);`
`205`	`205`	`header('Location: '.$targetUrl);`