Opened 15 years ago
Closed 12 years ago
#11679 closed Bug (invalid)
FormWizard: security_hash() has issues with Safari leaving in \r\n
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | contrib.formtools | Version: | 1.0 |
Severity: | Normal | Keywords: | FormWizard |
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In v1.0.3: The latest Safari is passing in \r\n within the text fields. These are blowing up the security_hash() call when on Step 2 of a multiple step FormWizard.
Try the test:
f1 = TestForm({'name': 'joe', 'bio': 'Nothing\r\nnotable.'})
Similar to fix r10752; changing django/contrib/formtools/utils.py as follows works around the issue:
25: if isinstance(value, basestring): 26: value = value.strip() 27+ value = value.replace('\r', ' ') 28+ value = value.replace('\n', ' ')
Change History (6)
comment:1 by , 15 years ago
Description: | modified (diff) |
---|
comment:2 by , 15 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:3 by , 14 years ago
Severity: | → Normal |
---|---|
Type: | → Bug |
comment:6 by , 12 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Fixed description: reference is to a changeset, not a ticket.