Feature #1482: Captive Portal support for a configurable RADIUS NAS ID - pfSense - pfSense bugtracker

Feature #1482

Captive Portal support for a configurable RADIUS NAS ID

Added by Richard Gate about 2 years ago. Updated 3 months ago.

Status:

Feedback

Start date:

04/26/2011

Priority:

Normal

Due date:

Assignee:

% Done:

Category:

Target version:

Affected version:

All

Affected Architecture:

Description

When running network traces to examine the packets sent/received during Captive Portal authentication and accounting with RADIUS,
I noticed that pfSense Captive Portal uses the local host and domain name for the RADIS NAS ID. I can see this is set in /etc/inc/cativeposrtl.inc
in the function getNasID().

Do you intend to make this a configurable option?
This would help users migrating from DD-WRT WiFiGator configurations, such as myself :-)
You could implement the code here - http://mike.eire.ca/2010/08/25/setting-nas-id-on-pfsense/

Thanks, Richard

nasid.patch - Patch against RELENG_2_0_1 (5.8 kB) Michael Newton, 03/11/2012 03:26 am

nasid.patch - Updated patch against RELENG_2_0_1 (5.8 kB) Michael Newton, 03/12/2012 06:12 pm

Associated revisions

Revision 9dcb92da
Added by Bill Marquette over 5 years ago

Ticket #1482 - set the source to an interface that is inside the subnet definition

Revision f971bb63
Added by Scott Ullrich over 5 years ago

IPSEC keep alive pinger using the wrong source IP address

Ticket #1482

History

#1 Updated by Chris Buechler about 2 years ago

Project changed from pfSense Packages to pfSense

#2 Updated by Chris Buechler about 2 years ago

Assignee deleted (~~Ermal Luçi~~)

#3 Updated by Richard Gate about 2 years ago

Hi just found out that the code in /etc/inc/radius.inc function putStandardAttributes() does not call the getNasId() function but works out the NAS ID for itself in the same way that getNasId() does. So it will need updating to call getNasId() in a similar way that getNasIp() is a few lines before before.
Hope this helps, Richard

#4 Updated by Jim P about 1 year ago

Status changed from New to Feedback

There have been NAS IP options for CP in the code base for a while now.

#5 Updated by Richard Gate about 1 year ago

BTW its "NAS ID" not "NAS IP".
Assuming that was a typo, where can you set the "NAS ID" in the CP configuration?

#6 Updated by Michael Newton about 1 year ago

File nasid.patch added

We're just putting together our first 2.0 boxes; it's currently untested, but I've attached the patch we're planning to try, based on our patch for 1.23 (that's my web page cited in the original report.) It would be nice to get this option added to the code.