fix: missing exception details in xblock-lti-consumer logs

This commit updates the version of the lti-consumer-xblock from 4.3.2 to 4.3.3. This installs the newest version of the lti-consumer-xblock library. This version includes the following changes.

The error handler in LtiConsumerXBlock.lti_1p3_launch_callback logs a warning when a select set of exceptions are handled. That log does not contain useful information about the nature of the exception, because the exceptions were not being instantiated with error messages. The try...catch is a large block that contains code that can raise a multitude of errors, so these changes will enable better debugging.

This commit:
* adds helpful messages to the raised exceptions.
* adds the "exc_info=True" argument to include the stack trace of the handled exception.
* adds ValueError and TypeError to the list of handled exceptions, because the code can raise exceptions of these types.
This commit is contained in:
michaelroytman
2022-08-03 08:17:25 -04:00
parent a948f535c1
commit 5d5fce2949
3 changed files with 3 additions and 3 deletions

View File

@@ -656,7 +656,7 @@ libsass==0.10.0
# ora2
loremipsum==1.0.5
# via ora2
lti-consumer-xblock==4.3.2
lti-consumer-xblock==4.3.3
# via -r requirements/edx/base.in
lxml==4.9.1
# via

View File

@@ -860,7 +860,7 @@ loremipsum==1.0.5
# via
# -r requirements/edx/testing.txt
# ora2
lti-consumer-xblock==4.3.2
lti-consumer-xblock==4.3.3
# via -r requirements/edx/testing.txt
lxml==4.9.1
# via

View File

@@ -818,7 +818,7 @@ loremipsum==1.0.5
# via
# -r requirements/edx/base.txt
# ora2
lti-consumer-xblock==4.3.2
lti-consumer-xblock==4.3.3
# via -r requirements/edx/base.txt
lxml==4.9.1
# via