Fixes ucfopen#727 by looking for LTI initiation in both POST and GET …

…per IMS spec
SimonRothUCF · Feb 17, 2022 · 167d1f1 · 167d1f1
1 parent 40b7cfd
commit 167d1f1
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/src/Controller/LtiController.php b/src/Controller/LtiController.php
@@ -263,10 +263,19 @@ protected function saveTokenToSession($token)
     protected function saveRequestToSession()
     {
         try {
+            $getParams = $this->request->query->all();
             $postParams = $this->request->request->all();
 
+            foreach ($getParams as $key => $val) {
+                if (!empty($val)) {
+                    $this->session->set($key, $val);
+                }
+            }
+
             foreach ($postParams as $key => $val) {
-                $this->session->set($key, $val);
+                if (!empty($val)) {
+                    $this->session->set($key, $val);
+                }
             }
 
             if (!$this->session->get('lms_api_domain')) {