Project

General

Profile

Bug #853

PPPoE Server radius config not applied

Added by Hadley Rich over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
PPPoE Server
Target version:
Start date:
08/27/2010
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.0
Affected Architecture:

Description

Just playing with pfSense (2.0-BETA4 (i386) built on Thu Aug 26 23:39:42 EDT 2010) and installed the freeradius package to run as a pppoe backend. It wasn't working and I discovered that the radius settings weren't being applied to the mpd conf file. This trivial patch fixes it for me;

--- vpn.inc.old    2010-08-27 21:21:30.000000000 +1200
+++ vpn.inc    2010-08-27 21:21:42.000000000 +1200
@@ -1274,9 +1274,9 @@
                     $mpdconf .= "    set ipcp dns " . join(" ", $syscfg['dnsserver']) . "\n";
             }

-            if (isset ($pppoecfg['radius']['enable'])) {
+            if (isset ($pppoecfg['radius']['server']['enable'])) {
                 $mpdconf .=<<<EOD
-    set radius server {$pppoecfg['server']['ip']} "{$pppoecfg['radius']['secret']}" 
+    set radius server {$pppoecfg['radius']['server']['ip']} "{$pppoecfg['radius']['server']['secret']}" 
     set radius retries 3
     set radius timeout 10
     set auth enable radius-auth

Associated revisions

Revision 37d7de2d (diff)
Added by Jim Pingle over 8 years ago

Fix some PPPoE server radius variable references. Fixes #853.

History

#1 Updated by Jim Pingle over 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#2 Updated by Chris Buechler over 8 years ago

  • Category set to PPPoE Server
  • Target version set to 2.0

#3 Updated by Hadley Rich over 8 years ago

Thanks for the super quick fix. I can confirm that this is working now.

There are a few other fields on that page which aren't applied to (ports, update interval, etc.), I'll see if I can sort those out too when I get a chance.

#4 Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to New

Ports for RADIUS and accounting are still not used

#5 Updated by Ermal Luçi over 8 years ago

  • Status changed from New to Feedback

#6 Updated by Ermal Luçi over 8 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF