Fix controller for exampleauth External authsource

tvdijen · tvdijen · commit 53d02fdd9fe4 · 2024-04-05T00:03:15.000+02:00
diff --git a/modules/exampleauth/routing/routes/routes.yml b/modules/exampleauth/routing/routes/routes.yml
@@ -5,7 +5,7 @@ exampleauth-authpage:
   defaults: {
     _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::authpage'
   }
-  methods: [POST]
+  methods: [GET, POST]
 
 exampleauth-redirecttest:
   path: /redirecttest
diff --git a/modules/exampleauth/src/Controller/ExampleAuth.php b/modules/exampleauth/src/Controller/ExampleAuth.php
@@ -72,7 +72,7 @@ public function authpage(Request $request): Template|RedirectResponse
          * Note that we don't actually validate the user in this example. This page
          * just serves to make the example work out of the box.
          */
-        $returnTo = $request->request->get('ReturnTo');
+        $returnTo = $request->get('ReturnTo');
         if ($returnTo === null) {
             throw new Error\Exception('Missing ReturnTo parameter.');
         }

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ exampleauth-authpage:`
`5`	`5`	`defaults: {`
`6`	`6`	`_controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::authpage'`
`7`	`7`	`}`
`8`		`- methods: [POST]`
	`8`	`+ methods: [GET, POST]`
`9`	`9`
`10`	`10`	`exampleauth-redirecttest:`
`11`	`11`	`path: /redirecttest`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ public function authpage(Request $request): Template\|RedirectResponse`
`72`	`72`	`* Note that we don't actually validate the user in this example. This page`
`73`	`73`	`* just serves to make the example work out of the box.`
`74`	`74`	`*/`
`75`		`- $returnTo = $request->request->get('ReturnTo');`
	`75`	`+ $returnTo = $request->get('ReturnTo');`
`76`	`76`	`if ($returnTo === null) {`
`77`	`77`	`throw new Error\Exception('Missing ReturnTo parameter.');`
`78`	`78`	`}`