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

Strict standards error with auth_radius

    Details

    • Testing Instructions:
      Hide

      (borrowed from MDL-37625)

      Requirements

      1. Debug developer enabled. To get any NOTICE/WARN...
      2. a RADIUS server
      3. the PHP RADIUS extension installed and available on the web server of your Moodle testing site

      Testing steps

      1. log into your Moodle test site as a system administrator
      2. configure the RADIUS auth plugin in Moodle by going to the following path in the Settings block – Site administration -> Plugins -> Authentication -> Manage authentication
      3. enable and configure the RADIUS server plugin so that it communicates with your RADIUS server
      4. log out of the Moodle site
      5. attempt to log into the Moodle site using the credentials for a test user from the RADIUS server
      6. you should be able to successfully log in and have a new account created for you if you have never logged in with that user before
      Show
      (borrowed from MDL-37625 ) Requirements Debug developer enabled. To get any NOTICE/WARN... a RADIUS server if you don't have one available you can setup one using FreeRADIUS – http://freeradius.org/ the PHP RADIUS extension installed and available on the web server of your Moodle testing site Testing steps log into your Moodle test site as a system administrator configure the RADIUS auth plugin in Moodle by going to the following path in the Settings block – Site administration -> Plugins -> Authentication -> Manage authentication enable and configure the RADIUS server plugin so that it communicates with your RADIUS server log out of the Moodle site attempt to log into the Moodle site using the credentials for a test user from the RADIUS server you should be able to successfully log in and have a new account created for you if you have never logged in with that user before
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      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.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    13/May/13