search

moodle-saml / auth

Moodle SAML Auth plugin
https://moodle.org/plugins/auth_saml
3 stars 21 forks source link

SAML-14: Fixed fatal error due to course shortname. #17

Closed tbannister closed 5 years ago

tbannister commented 5 years ago

A fatal error could be triggered by course shortnames with characters that are valid in shortnames but not valid in admin settings. Invalid characters will now be filtered out.

tbannister commented 5 years ago

I made a small update to the previous commit, since one of our support staff ran into a issue with course shortnames that were too long. I added a substr call to limit the admin setting to using the first 60 characters of the string after the invalid characters have been stripped out.

tbannister commented 5 years ago

This issue has now been addressed by a July 16th commit that took a more expansive approach to fixing the problem.