Server error when access is denied due to an access change

numbas / numbas-lti-provider

An LTI tool provider to run Numbas exams

https://docs.numbas.org.uk/lti/en/latest/

Other

11 stars 12 forks source link

Server error when access is denied due to an access change #257

Closed christianp closed 1 year ago

christianp commented 1 year ago

We had a report from a student affected by an access change that they saw "Server error 500" instead of a nice explanation of why they couldn't access a resource.

christianp commented 1 year ago

The Server Error is only shown when the student hasn't made an attempt yet.

christianp commented 1 year ago

This was a problem in the patch_reverse method of django_auth_lti: it wasn't checking that request.LTI is defined. This wasn't a problem until I started building a URL for SEB in the context processor.