-
Bug
-
Resolution: Fixed
-
Minor
-
2.3.3, 2.4.1, 2.5
-
MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
-
MOODLE_23_STABLE, MOODLE_24_STABLE
-
This is a followup of MDL-37625.
When it was being tested this strict standards notice was shown:
Strict Standards: Non-static method PEAR::loadExtension() should not
|
be called statically, assuming $this from incompatible context in
|
lib/pear/Auth/RADIUS.php on line 49
|
And this was commented there:
it seems that the class has not been changed as trunk continue showing that static behavior:
http://svn.php.net/viewvc/pear/packages/Auth_RADIUS/trunk/RADIUS.php?revision=257341&view=markup
So, or we fix that in the PEAR wrapper or we consider switching to another (pure or wrapper) lib. If this had to be voted, I'd say "pure" is better (less dependencies). Like, for example:
http://www.phpclasses.org/package/4326-PHP-Authenticate-users-with-a-RADIUS-server.html
Although I don't know much really. Adding Jonathan Harker here coz he did the initial (and current) implementation.
- Testing discovered
-
MDL-37625 Included PEAR Crypt_CHAP library uses obsolete mhash() code which breaks RADIUS auth support in some PHP 5.3 installs
-
- Closed
-