There is wrong code in:
src/Controller/LTIToolProviderV1P3Launch.php
modules/lti_tool_provider_content/src/Controller/LTIToolProviderContentLaunch.php
code such as: "return new RedirectResponse('/', 500);"
Is erroneous and will not work. 500 is not a valid redirect code, and you will get a 500 error because of this code (not the response desired).
RedirectResponse can take other codes, such as 302.
I do not know if this is the kind of behavior desired, but one could replace these with:
- return new RedirectResponse('/', 500);
+ $accessDeniedUrl = Url::fromRoute('system.403')->toString();
+ return new RedirectResponse($accessDeniedUrl, 302);
If a 500 response is necessary, I think you may need to issue http_response_code(500);, then print something to the screen, and exit.
Active
2.0
Code
Hi @bob.hinrichs,
I have created MR!3 for the above issue, Please review it once.
I've created MR!4 which applies this same fix also in src/Controller/LTIToolProviderV1P3Launch.php as @bob.hinrichs reported. Please review, thank you!