PacketFence
Bug Tracking System

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001210PacketFenceerror-handlingpublic2011-04-21 08:422014-04-24 11:58
Reporterjamest 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusconfirmedResolutionopen 
PlatformOSOS Version
Product Version2.1.0 
Target Version3.6.1Fixed in Version 
Summary0001210: Can't add/edit person with pfcmd if firstname or lastname have spaces in them
DescriptionIf you try and add a person with pfcmd person add pid and have an assignment of firstname or surname with the add, this will fail with grammar test failed if the name has a space in it. The same thing happens with edit.
There's nothing I could see in the admin guide saying that names with spaces in are invalid, they can be added directly into the database with no apparent ill effect.
I would suggest that either pfcmd is fixed to allow these names (as people do have them in real life), or the restriction is documented and pfcmd gives a more meaningful error message.
Additional InformationSome example output:
./pfcmd person add test5 firstname="space in the middle"
Command not understood. (pfcmd grammar test failed at line 223.)
Usage: pfcmd person <add|view|edit|delete> pid [assignments]

manipulate person entries

examples:
  pfcmd person view all
  pfcmd person add bjenkins notes="Bob Jenkins"
  pfcmd person delete bjenkins

./pfcmd person add test5 surname="space in the middle"
Command not understood. (pfcmd grammar test failed at line 223.)
Usage: pfcmd person <add|view|edit|delete> pid [assignments]

manipulate person entries

examples:
  pfcmd person view all
  pfcmd person add bjenkins notes="Bob Jenkins"
  pfcmd person delete bjenkins

./pfcmd person add test5 firstname="underscore_in_the_middle"
TagsNo tags attached.
fixed in git revision
fixed in mtn revision
Attached Files

- Relationships

-  Notes
(0003000)
obilodeau (reporter)
2012-08-31 10:34

This should be fixed now. I'll try to reproduce later in the lab and let you know.
(0003002)
obilodeau (reporter)
2012-08-31 10:48

In stable this isn't fixed now but 0001523 should fix it. Let me check that.
(0003003)
obilodeau (reporter)
2012-08-31 10:58

No it's not :(
(0003007)
obilodeau (reporter)
2012-08-31 15:13

Debugging the parser, I don't really understand what's going on here..

I was able to make it work with:
pfcmd 'person add bjenkins notes="Bob Jenkins"'

I'll have to compare the parser debug results but this implies that it works from the Web Admin since it always single quotes all arguments together.
(0003175)
fgaudreault (viewer)
2012-10-19 13:48

Need to see if it's fixed or not.
(0003237)
fgaudreault (viewer)
2012-10-24 10:04

Still an issue. Tested on 3.6.0-devel.

[root@pf-3-centos6 pf]# ./bin/pfcmd person add test5 firstname="space in the middle"
Command not understood. (pfcmd grammar test failed at line 210.)
Usage: pfcmd person <add|view|edit|delete> pid [assignments]

- Issue History
Date Modified Username Field Change
2011-04-21 08:42 jamest New Issue
2012-08-31 10:32 obilodeau Status new => assigned
2012-08-31 10:32 obilodeau Assigned To => obilodeau
2012-08-31 10:34 obilodeau Note Added: 0003000
2012-08-31 10:48 obilodeau Note Added: 0003002
2012-08-31 10:58 obilodeau Note Added: 0003003
2012-08-31 15:13 obilodeau Note Added: 0003007
2012-10-19 13:48 fgaudreault Assigned To obilodeau =>
2012-10-19 13:48 fgaudreault Target Version => investigate
2012-10-19 13:48 fgaudreault Note Added: 0003175
2012-10-24 10:04 fgaudreault Note Added: 0003237
2012-10-24 10:04 fgaudreault Status assigned => confirmed
2012-10-24 10:04 fgaudreault Target Version investigate => 3.6.1


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker