View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
11090 | Bug reports | Survey participants (Tokens) | public | 2016-04-30 11:19 | 2016-05-17 16:04 |
Reporter | madflow | Assigned To | c_schmitz | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.06+ | ||||
Fixed in Version | 2.50.x | ||||
Summary | 11090: Import sanitizes Tokens / Manual Entry does not | ||||
Description | If you create a token via the web interface, tokens like 'kb2n#t8i%po%qp' are valid. If you import these tokens, they are sanitized with Token::sanitizeToken($token) and result in 'kb2nt8ipoqp' in the database. The user is not informed about this and there obviously the software yields different results how the user creates these token. The expected behaviour is: Preferred: Never sanititze! Just define a min/max length. There are use cases where the tokens are handed out to the participants and not emailed to them. Szenario 1: Treat "invalid" tokens always the same (web interface, import) do not allow invalid tokens. Szenario 2: Always sanitize the tokens - but inform the user. | ||||
Steps To Reproduce | Create the the token above via the webinterface or import it. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | master | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | ALL | ||||
Database type & version | ALL | ||||
Server OS (if known) | ALL | ||||
Webserver software & version (if known) | ALL | ||||
PHP Version | ALL | ||||
Hi, there are a bug + a feature request here. No ? |
|
Thanks for getting back to me on this. Well - I guess that they are treated differently and there is no info for the user is a bug. The "Preferred" szenario is a feature request. In my opinion there is no use in fixing a bug, that fosters "wrong" behaviour. |
|
This is resolved in the latest version so it works consistently. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2016-04-30 11:19 | madflow | New Issue | |
2016-04-30 16:03 | DenisChenu | Note Added: 38033 | |
2016-04-30 17:02 | madflow | Note Added: 38059 | |
2016-04-30 17:02 | madflow | Note Edited: 38059 | |
2016-05-17 16:04 | c_schmitz | Note Added: 38638 | |
2016-05-17 16:04 | c_schmitz | Status | new => closed |
2016-05-17 16:04 | c_schmitz | Assigned To | => c_schmitz |
2016-05-17 16:04 | c_schmitz | Resolution | open => fixed |
2016-05-17 16:04 | c_schmitz | Fixed in Version | => 2.5 |
2016-12-08 10:39 | c_schmitz | Category | Tokens => Survey participants (Tokens) |