PacketFence - BTS - PacketFence
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001253PacketFencecorepublic2011-08-24 19:392012-06-14 12:16
ReporterTom Fischer 
Assigned Toobilodeau 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version2.2.1 
Target Version3.4.0Fixed in Version3.4.0 
fixed in git revision3584e75
fixed in mtn revision
Summary0001253: backslashes not handled properly in pid field
Description   I am testing PF 2.2.1 using the Vmplayer version. I have it setup to authenticate my wireless connections to my AD domain. The connection and VLAN assignment work well, but the node is tagged with the identifier as "domain\\user". The person Identifier is in the same format. When I try to edit either, I get:

Report BugWarning: Invalid argument supplied for foreach() in /usr/local/pf/html/admin/person/lookup.php on line 39

Error: Problems executing 'PFCMD person view domain\\user'

Command not understood. (pfcmd grammar test failed at line 227.)

   Can I change that identifier field to Stripped-Username somewhere? Or can we get PF to handle the backslashes? The only way that I have found to get rid of these is to drop and re-create the database.

Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-08-24 19:39Tom FischerNew Issue
2011-08-25 12:26Tom FischerNote Added: 0002160
2011-09-19 16:26obilodeauStatusnew => assigned
2011-09-19 16:26obilodeauAssigned To => obilodeau
2011-09-19 16:26obilodeauSeveritymajor => minor
2011-09-19 16:26obilodeauCategorylogging => core
2011-09-19 16:26obilodeauTarget Version => +1
2011-09-19 16:26obilodeauDescription Updated
2012-05-08 10:34obilodeauNote Added: 0002707
2012-05-30 11:45obilodeauNote Added: 0002737
2012-06-07 10:58obilodeauSummarybackslashes not handles properly when inserted into database => backslashes not handled properly in pid field
2012-06-07 11:06obilodeauNote Added: 0002745
2012-06-11 11:30obilodeaugit revision => 3584e75
2012-06-11 11:30obilodeauNote Added: 0002752
2012-06-11 11:30obilodeauStatusassigned => resolved
2012-06-11 11:30obilodeauFixed in Version => +1
2012-06-11 11:30obilodeauResolutionopen => fixed
2012-06-14 12:15obilodeauTarget Version+1 => 3.4.0
2012-06-14 12:15obilodeauFixed in Version+1 => 3.4.0
2012-06-14 12:16obilodeauNote Added: 0002778
2012-06-14 12:16obilodeauStatusresolved => closed

Notes
(0002160)
Tom Fischer   
2011-08-25 12:26   
I guess we need to generalize this more. I copied the radius.pm to custom.pm so that it would use the stripped username. There is also a problem with spaces in the username, ala, "firstname lastname".
(0002707)
obilodeau   
2012-05-08 10:34   
Reminder sent to: fgaudreault

What do you think about this one Francois? Is this fixed now or is there something simple we could do?
(0002737)
obilodeau   
2012-05-30 11:45   
A fix to allow domain\\user up to the core is on the way.
(0002745)
obilodeau   
2012-06-07 11:06   
See https://github.com/inverse-inc/packetfence/pull/26 [^]
(0002752)
obilodeau   
2012-06-11 11:30   
Fix is in stable. Will be part of next release.
(0002778)
obilodeau   
2012-06-14 12:16   
fix released in 3.4.0 yesterday