PacketFence - BTS - PacketFence
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001051PacketFenceradiuspublic2010-08-25 16:272012-08-14 09:15
Reporterobilodeau 
Assigned Toobilodeau 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.0.0Fixed in Version3.0.0 
fixed in git revision
fixed in mtn revision
Summary0001051: radiusd restarted by PacketFence
DescriptionBased on the current way we want to tackle 0001028, we will need to restart radiusd whenever the switch configuration changes.

See http://github.com/alandekok/freeradius-server/blob/v2.1.x/raddb/sql.conf [^] for details:
        # Clients will ONLY be read on server startup. For performance
        # and security reasons, finding clients via SQL queries CANNOT
        # be done "live" while the server is running.
        #

This bug is to track this change.
Steps To Reproduce
Additional Information
TagsCode Review
Relationships
related to 0001028closed obilodeau adding radius clients needs to be performed from the command line 
related to 0001027closed fgaudreault freeradius needs to be configured manually 
Attached Filespatch 001051.patch (7,699) 2011-05-06 12:33
https://www.packetfence.org/bugs/file_download.php?file_id=93&type=bug
Issue History
Date ModifiedUsernameFieldChange
2010-08-25 16:27obilodeauNew Issue
2010-08-25 16:27obilodeauStatusnew => assigned
2010-08-25 16:27obilodeauAssigned To => obilodeau
2010-08-25 16:27obilodeauRelationship addedchild of 0001027
2010-08-25 16:28obilodeauRelationship addedrelated to 0001028
2010-08-25 17:50ludovicNote Added: 0001635
2010-11-19 14:25obilodeauTarget Version1.10.0 => 2.0.0
2011-01-18 09:30obilodeauTarget Version2.0.0 => 2.1.0
2011-03-03 15:16obilodeauTarget Version2.1.0 => +1
2011-03-03 15:18obilodeauTarget Version+1 => +2
2011-03-17 14:06fgaudreaultRelationship deletedchild of 0001027
2011-03-17 14:06fgaudreaultRelationship addedrelated to 0001027
2011-05-05 10:34fgaudreaultAssigned Toobilodeau => fgaudreault
2011-05-06 12:32fgaudreaultNote Added: 0002060
2011-05-06 12:33fgaudreaultTag Attached: Code Review
2011-05-06 12:33fgaudreaultStatusassigned => feedback
2011-05-06 12:33fgaudreaultFile Added: 001051.patch
2011-05-06 12:34fgaudreaultNote Edited: 0002060
2011-05-06 12:35fgaudreaultNote Edited: 0002060
2011-05-06 12:47fgaudreaultNote Edited: 0002060
2011-05-16 15:51fgaudreaultNote Added: 0002067
2011-05-19 16:48fgaudreaultNote Added: 0002068
2011-05-19 16:48fgaudreaultStatusfeedback => assigned
2011-05-19 16:48fgaudreaultAssigned Tofgaudreault => obilodeau
2011-05-19 16:57fgaudreaultNote Edited: 0002068
2011-08-11 10:11obilodeauNote Added: 0002119
2011-08-24 16:38obilodeauNote Added: 0002156
2011-08-24 16:38obilodeauStatusassigned => resolved
2011-08-24 16:38obilodeauFixed in Version => trunk
2011-08-24 16:38obilodeauResolutionopen => fixed
2011-09-21 22:07obilodeauFixed in Versiontrunk => 3.0.0
2011-09-21 22:15obilodeauNote Added: 0002264
2011-09-21 22:16obilodeauStatusresolved => closed
2011-10-25 09:06obilodeauTarget Version+2 => 3.0.0
2012-08-14 09:15obilodeauCategoryradius module => radius

Notes
(0001635)
ludovic   
2010-08-25 17:50   
http://wiki.freeradius.org/index.php/FAQ#How_do_I_get_radius_to_pick_up_changes_in_the_raddb.2Fusers_file.3F [^]

http://wiki.freeradius.org/index.php/FAQ#How_do_I_check_the_configuration_before_sending_a_HUP_to_the_server.3F [^]

Might be worth a read.
(0002060)
fgaudreault   
2011-05-06 12:32   
(edited on: 2011-05-06 12:47)
First step toward this, here is a patch.

Missing from this patch, the "reload" option for freeradius. It adds two new fields called vlan.radiusd and the services.radiusd in pf.conf
(0002067)
fgaudreault   
2011-05-16 15:51   
kill -1 does not reload nas clients from the db. The only option right now is to restart radius completely to reload the nas table.

Only those modules are reloaded with the HUP :
Mon May 16 15:47:32 2011 : Info: Received HUP.
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "files"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "suffix"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "attr_filter.access_reject"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "attr_filter.accounting_response"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "pap"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "radutmp"
Mon May 16 15:47:32 2011 : Info: Module: Reloaded module "detail"

usersfile = ${confdir}/users
acctusersfile = ${confdir}/acct_users
preproxy_usersfile = ${confdir}/preproxy_users
(0002068)
fgaudreault   
2011-05-19 16:48   
(edited on: 2011-05-19 16:57)
This is done in org.packetfence.feature.radius-accounting

Rev : cc98aa703270fb9c953d2ce0ab9557d49e139b67

To be reviewed prior the release
(0002119)
obilodeau   
2011-08-11 10:11   
Reminder sent to: fgaudreault

Do we want this in 3.0?
(0002156)
obilodeau   
2011-08-24 16:38   
freeradius (radiusd) is now a service managed by PacketFence
(0002264)
obilodeau   
2011-09-21 22:15   
fix released in 3.0