Ticket #287 (closed Bug report: fixed)

Opened 14 years ago

Last modified 14 years ago

pear RADIUS dependency

Reported by: Thus0 Owned by:
Priority: high Milestone:
Component: Auth server, Other Version:
Keywords: radius Cc:

Description

Hi,

with rev 1151, radius backend does not seem to work anymore because of changes in classes/Dependencies.php

I had to change 2 files to make it work again :

classes/Dependencies.php

@@ -127,7 +127,7 @@
            "detectFiles" => "HTML/Safe.php", 
            'description' => "Optional for content type Langstring (and subtypes) to better strip out dangerous HTML"
            ),
-       "Radius" => array (
+       "RADIUS" => array (
            "type" => "pearStandard", 
            'description' => "Required by the optional Radius Authenticator"
            ),

classes/Authenticators/AuthenticatorRadius.php

@@ -133,7 +133,7 @@
         // Call parent constructor
         parent :: __construct($account_orgin);
 
-        if (Dependencies::check("Radius")) {
+        if (Dependencies::check("RADIUS")) {
             /**
              * Including PEAR RADIUS and CHAP MD5 interface classes
              */
@@ -180,7 +180,7 @@
         $username = $db->escapeString($username);
         $password = $db->escapeString($password);
 
-        if (Dependencies::check("Radius", $errmsg)) {
+        if (Dependencies::check("RADIUS", $errmsg)) {
             /*
              * Supported encryption methods are :
              *
@@ -313,7 +313,7 @@
         // Init values
         $info = null;
 
-        if (Dependencies::check("Radius", $errmsg)) {
+        if (Dependencies::check("RADIUS", $errmsg)) {
             $conn_id = $db->escapeString($conn_id);
             $db->execSqlUniqueRes("SELECT CURRENT_TIMESTAMP, *, CASE WHEN ((CURRENT_TIMESTAMP - reg_date) > networks.validation_grace_time) THEN true ELSE false END AS validation_grace_time_expired FROM connections JOIN users ON (users.user_id=connections.user_id) JOIN networks ON (users.account_origin = networks.network_id) WHERE connections.conn_id=$conn_id", $info, false);
 
@@ -382,7 +382,7 @@
         $info = null;
         $conn_id = $db->escapeString($conn_id);
 
-        if (Dependencies::check("Radius", $errmsg)) {
+        if (Dependencies::check("RADIUS", $errmsg)) {
             $db->execSqlUniqueRes("SELECT CURRENT_TIMESTAMP, *, CASE WHEN ((CURRENT_TIMESTAMP - reg_date) > networks.validation_grace_time) THEN true ELSE false END AS validation_grace_time_expired FROM connections JOIN users ON (users.user_id=connections.user_id) JOIN networks ON (users.account_origin = networks.network_id) WHERE connections.conn_id=$conn_id", $info, false);

             // RADIUS accounting ping
@@ -452,7 +452,7 @@
         $info = null;
         $conn_id = $db->escapeString($conn_id);
 
-        if (Dependencies::check("Radius", $errmsg)) {
+        if (Dependencies::check("RADIUS", $errmsg)) {
             $db->execSqlUniqueRes("SELECT CURRENT_TIMESTAMP, *, CASE WHEN ((CURRENT_TIMESTAMP - reg_date) > networks.validation_grace_time) THEN true ELSE false END AS validation_grace_time_expired FROM connections JOIN users ON (users.user_id=connections.user_id) JOIN networks ON (users.account_origin = networks.network_id) WHERE connections.conn_id=$conn_id", $info, false);

-- Thus0

Change History

Changed 14 years ago by benoitg

  • status changed from new to closed
  • resolution set to fixed

Fixed in [1166]

Note: See TracTickets for help on using tickets.