Anonymous | Login | 2024-08-24 22:17 EDT | ![]() |
Main | My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | Project | Category | View Status | Date Submitted | Last Update | |||
0001064 | PacketFence | error-handling | public | 2010-09-08 14:25 | 2015-02-18 10:55 | |||
Reporter | obilodeau | |||||||
Assigned To | ||||||||
Priority | normal | Severity | minor | Reproducibility | sometimes | |||
Status | closed | Resolution | open | |||||
Platform | OS | OS Version | ||||||
Product Version | ||||||||
Target Version | Fixed in Version | |||||||
Summary | 0001064: usernames with strange characters were accepted by pf::web but not by `bin/pfcmd manage register...` | |||||||
Description | The end effect was that users got the message where they successfully registered (with release page) but since `pfcmd manage register...` failed, it stayed in registration and got the captive portal again after the release page finished. Not so sure what should be done here.. but since auth returned true, I would accept them. But since we can't predict what will be accepted by auth (we rely it to a 3rd party), we should maybe do a first pass username validation, then auth then pfcmd manage register. problematic characters include: _, [...], ` | |||||||
Tags | No tags attached. | |||||||
fixed in git revision | ||||||||
fixed in mtn revision | ||||||||
Attached Files | ||||||||
![]() |
|
(0003898) lmunro (administrator) 2015-02-18 10:55 |
Obsolete bug tracker entries. PF 4 introduced changes that either make these irrelevant or impossible to reproduce. New issues are moving to github issues. |
![]() |
|||
Date Modified | Username | Field | Change |
2010-09-08 14:25 | obilodeau | New Issue | |
2010-09-22 16:02 | obilodeau | Target Version | 1.9.2 => 1.9.3 |
2012-10-19 13:38 | fgaudreault | Target Version | 1.9.3 => general |
2015-02-18 10:55 | lmunro | Note Added: 0003898 | |
2015-02-18 10:55 | lmunro | Status | new => closed |
Copyright © 2000 - 2012 MantisBT Group |