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

[raeburn]

raeburn		Tue Feb  8 15:08:53 2022 EDT

  Modified files:              
    /loncom/lti	ltiauth.pm 
  Log:
  - Bug 6907
    - Modify message logged for user session launched from LTI-protected 
      deep-link which includes username in signed payload. 
    - On a balancer call lonauth::success before redirect to switchserver
    - Fix typo.   
  
  
Index: loncom/lti/ltiauth.pm
diff -u loncom/lti/ltiauth.pm:1.32 loncom/lti/ltiauth.pm:1.33
--- loncom/lti/ltiauth.pm:1.32	Sun Feb  6 21:37:00 2022
+++ loncom/lti/ltiauth.pm	Tue Feb  8 15:08:53 2022
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Basic LTI Authentication Module
 #
-# $Id: ltiauth.pm,v 1.32 2022/02/06 21:37:00 raeburn Exp $
+# $Id: ltiauth.pm,v 1.33 2022/02/08 15:08:53 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1054,13 +1054,14 @@
     my ($r,$uname,$cnum,$cdom,$uhome,$itemid,$ltitype,$dest,$lonhost) = @_;
     $r->user($uname);
     if ($ltitype eq 'c') {
-        &Apache::lonnet::logthis("Course Link Protector ($itemid) authorized student: $uname:$cdom, course: $cdom\_$cnum");
+        &Apache::lonnet::logthis("Link Protector: $itemid (c) authorized student: $uname:$cdom, course: $cdom\_$cnum");
     } elsif ($ltitype eq 'd') {
-        &Apache::lonnet::logthis("Domain LTI for link protection ($itemid) authorized student: $uname:$cdom, course: $cdom\_$cnum");
+        &Apache::lonnet::logthis("Link Protector: $itemid (d) authorized student: $uname:$cdom, course: $cdom\_$cnum");
     }
     my ($is_balancer,$otherserver,$hosthere) = &check_balancer($r,$uname,$cdom);
     if (($is_balancer) && (!$hosthere)) {
         # login but immediately go to switch server
+        &Apache::lonauth::success($r,$uname,$cdom,$uhome,'noredirect');
         $env{'form.origurl'} = $dest;
         $env{'request.linkprot'} = $itemid.$ltitype.':'.$dest;
         $env{'request.deeplink.login'} = $dest;
@@ -1128,7 +1129,7 @@
         &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:').
         '</p>'.
-        '<p class="LC_error">'.$msga'.</p>'.
+        '<p class="LC_error">'.$msg.'</p>'.
         &Apache::loncommon::end_page());
     return;
 }