fix: address feedback, log request URL instead

Author: arslanashraf7

lms/djangoapps/lti_provider/users.py

@@ -60,32 +60,32 @@ def authenticate_lti_user(request, lti_user_id, lti_consumer):
             log.info(
                 'LTI consumer requires existing user account for LTI user ID: %s from request: %s',
                 lti_user_id,
-                request
+                request.path
             )
             if request.user.is_authenticated and request.user.email.lower() == profile["email"]:
                 lti_user = create_lti_user(lti_user_id, lti_consumer, profile)
             else:
                 log.error(
-                    'LTI user account linking failed for LTI user ID: %s from request: %s: '
+                    'LTI user account linking failed for LTI user ID: %s for request: %s: '
                     'either user is not logged in or email mismatched',
                     lti_user_id,
-                    request
+                    request.path
                 )
                 # Ask the user to login before linking.
                 raise PermissionDenied() from exc
         elif lti_consumer.use_lti_pii:
             log.info(
                 'Creating LTI user with PII for LTI user ID: %s from request: %s',
                 lti_user_id,
-                request
+                request.path
             )
             profile["username"] = lti_user_id
             lti_user = create_lti_user(lti_user_id, lti_consumer, profile)
         else:
             log.info(
                 'Creating LTI user without PII for LTI user ID: %s from request: %s',
                 lti_user_id,
-                request
+                request.path
             )
             lti_user = create_lti_user(lti_user_id, lti_consumer)
 
@@ -96,7 +96,7 @@ def authenticate_lti_user(request, lti_user_id, lti_consumer):
         log.info(
             'Switching logged-in user to LTI user ID: %s for request: %s',
             lti_user_id,
-            request
+            request.path
         )
         switch_user(request, lti_user, lti_consumer)
 
@@ -161,7 +161,7 @@ def switch_user(request, lti_user, lti_consumer):
     if not edx_user:
         # This shouldn't happen, since we've created edX accounts for any LTI
         # users by this point, but just in case we can return a 403.
-        log.error('Switching user failed for LTI user ID: %s from request: %s', lti_user.lti_user_id, request)
+        log.error('Switching user failed for LTI user ID: %s from request: %s', lti_user.lti_user_id, request.path)
         raise PermissionDenied()
     login(request, edx_user)
     mark_user_change_as_expected(edx_user.id)

lms/djangoapps/lti_provider/views.py

@@ -63,7 +63,7 @@ def lti_launch(request, course_id, usage_id):
     # missing
     params = get_required_parameters(request.POST)
     if not params:
-        log.info('Missing required LTI parameters in LTI request: %s', request)
+        log.info('Missing required LTI parameters in LTI request: %s', request.path)
         return HttpResponseBadRequest()
     params.update(get_optional_parameters(request.POST))
     params.update(get_custom_parameters(request.POST))
@@ -81,15 +81,15 @@ def lti_launch(request, course_id, usage_id):
             'consumer key: %s and instance GUID: %s for request: %s',
             params['oauth_consumer_key'],
             params.get('tool_consumer_instance_guid', None),
-            request
+            request.path
         )
         return HttpResponseForbidden()
 
     # Check the OAuth signature on the message
     if not SignatureValidator(lti_consumer).verify(request):
         log.error(
             'Invalid OAuth signature for LTI launch from request: %s',
-            request
+            request.path
         )
         return HttpResponseForbidden()
 
@@ -101,7 +101,7 @@ def lti_launch(request, course_id, usage_id):
             'Invalid course key %s or usage key %s from request %s',
             course_id,
             usage_id,
-            request
+            request.path
         )
         raise Http404()  # lint-amnesty, pylint: disable=raise-missing-from
     params['course_key'] = course_key
@@ -116,7 +116,7 @@ def lti_launch(request, course_id, usage_id):
         log.info(
             'LTI user authentication failed for user Id: %s from request: %s',
             user_id,
-            request
+            request.path
         )
         request.session.flush()
         context = {