[LON-CAPA-cvs] cvs: loncom /lti ltiauth.pm

[raeburn]

raeburn		Wed Apr  6 21:44:49 2022 EDT

  Modified files:              
    /loncom/lti	ltiauth.pm 
  Log:
  - Bug 6907
   - Don't show log-in page if link protection configuration has 
     "Action when username is not for an enrolled student" set to: 
     "Discontinue launch process", and username was not available.
   - White space added between two sentences in displayed warning.
  
  
Index: loncom/lti/ltiauth.pm
diff -u loncom/lti/ltiauth.pm:1.35 loncom/lti/ltiauth.pm:1.36
--- loncom/lti/ltiauth.pm:1.35	Tue Mar 29 19:37:25 2022
+++ loncom/lti/ltiauth.pm	Wed Apr  6 21:44:49 2022
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Basic LTI Authentication Module
 #
-# $Id: ltiauth.pm,v 1.35 2022/03/29 19:37:25 raeburn Exp $
+# $Id: ltiauth.pm,v 1.36 2022/04/06 21:44:49 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -233,6 +233,7 @@
                                 }
                                 if ($lti_in_use{'notstudent'} eq 'reject') {
                                     &invalid_request($r,'Information for valid user missing from launch request');
+                                    return OK;
                                 }
                             }
                         }
@@ -1128,7 +1129,7 @@
         '<h3>'.&mt('Invalid LTI launch request').'</h3>'.
         '<p class="LC_warning">'.
         &mt('Launch of LON-CAPA is unavailable from the "external tool" link you had followed in another web application.').
-        &mt('Launch failed for the following reason:').
+        ' '.&mt('Launch failed for the following reason:').
         '</p>'.
         '<p class="LC_error">'.$msg.'</p>'.
         &Apache::loncommon::end_page());