Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-19671

username from phpCAS::getUser() can be mixed case and we check against a lower-case value

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.9, 1.9.5, 2.0
    • Fix Version/s: 1.8.10, 1.9.6, 2.0
    • Component/s: Authentication
    • Labels:
      None
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      As Julien Gribonvald points out in http://moodle.org/mod/forum/discuss.php?d=126988 we should use moodle_strtolower on the returned value before comparing both values.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            iarenaza Iñaki Arenaza added a comment -

            Fixed in 1.8, 1.9 and HEAD.

            Thanks a lot to Julien Gribonvald for the bug report and suggested fix

            Saludos.
            Iñaki.

            Show
            iarenaza Iñaki Arenaza added a comment - Fixed in 1.8, 1.9 and HEAD. Thanks a lot to Julien Gribonvald for the bug report and suggested fix Saludos. Iñaki.

              People

              • Assignee:
                iarenaza Iñaki Arenaza
                Reporter:
                iarenaza Iñaki Arenaza
                Participants:
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  21/Oct/09