Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major 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.

Activity

Hide
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
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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: