Closed
Bug 1273538
Opened 9 years ago
Closed 9 years ago
Encode white spaces with '%20' instead of '+'
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: adrian, Assigned: adrian)
Details
Attachments
(2 files)
Bug 1270662 has broken some of our URLs, notably SuperSearch based ones, because we stopped replacing the plus sign (+) with white spaces. However, most of our URL encoding does encode spaces with pluses. We should instead encode spaces with %20.
| Assignee | ||
Comment 1•9 years ago
|
||
Comment 2•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/8b9cea756d87ddadf1a3e4909da379ed7584458e
Fixes bug 1273538 - Correctly encode white spaces in URLs.
https://github.com/mozilla/socorro/commit/0680f3170c246de215332eadf45461704aeb31fa
Merge pull request #3348 from adngdb/1273538-correctly-encode-white-space
Fixes bug 1273538 - Correctly encode white spaces in URLs.
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 3•9 years ago
|
||
On [1], when you click on '2' to go to page 2, there is no results because spaces are replaced by '+' in the signature.
[1] https://crash-stats.mozilla.com/search/?product=Firefox&release_channel=beta&signature=%3Dshutdownhang%20%7C%20_VEC_memset&date=%3E%3D2016-06-21&_sort=-date&_facets=signature&_facets=build_id&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#crash-reports
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
| Assignee | ||
Comment 4•9 years ago
|
||
Comment 5•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/4c72165a51a28b37f0f1b00e7bbe27e1335af672
Fixes bug 1273538 - Fixes white spaces encoding in pagination URLs. (#3395)
* Fixes bug 1273538 - Fixes white spaces encoding in pagination URLs.
* refactoring
* New name.
Updated•9 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•