From 69cc0f3c25bf71bf39c325fb1670c5c0a9197d65 Mon Sep 17 00:00:00 2001 From: Sebastian Serth Date: Wed, 20 Sep 2023 23:53:35 +0200 Subject: [PATCH] Retry storing LTI data on invalid record We might receive a `RecordInvalid` error, if one part (such as the external user) is already assigned (not unique). Hence, we just retry. Fixes CODEOCEAN-TZ --- app/controllers/concerns/lti.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/concerns/lti.rb b/app/controllers/concerns/lti.rb index 53066ae0..6437b731 100644 --- a/app/controllers/concerns/lti.rb +++ b/app/controllers/concerns/lti.rb @@ -226,7 +226,7 @@ module Lti @lti_parameters.save! session[:external_user_id] = current_user.id - rescue ActiveRecord::RecordNotUnique + rescue ActiveRecord::RecordNotUnique, ActiveRecord::RecordInvalid retry end