805597 - [socorro-crashstats] 500 Internal Server Error (null-byte) on report/list

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

http://crash-stats-new-dev.allizom.org/report/list?signature=nsXULPopupManager%3A%3AShowTooltipAtScreen%28nsIContent%2A%2C+nsIContent%2A%2C+int%2C+int%29%00%27%22--%3E%3C%2Fstyle%3E%3C%2Fscript%3E%3Cscript%3Enetsparker(0x0005BA)%3C%2Fscript%3E&product=Firefox&query_type=is_exactly&range_unit=hours&process_type=any&platform=windows&hang_type=any&date=2012-10-25+18%3A49%3A34&range_value=1 throws a 500 in the webapp, but it's actually a null-byte problem, which should, I think, return a 404?

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Try http://crash-stats-new-dev.allizom.org/report/list?range_unit=days&version=Firefox%3A16.0.2&range_value=7&signature=http%3a%2f%2fnetsparker.com%2fn%3f%00.php&page=11, now.

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro-crashstats

https://github.com/mozilla/socorro-crashstats/commit/3a32d34d3f4d2467a28249e8c49371a883159b8a
Fixes bug 805597 - Removed null bytes from middleware URL.

https://github.com/mozilla/socorro-crashstats/commit/3cf38235d88eca64fcd0d53605d75ae6246c124b
Merge pull request #267 from AdrianGaudebert/805597-remove-null-bytes-from-mware-urls

Fixes bug 805597 - Removed null bytes from middleware URL.

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Hrm, http://crash-stats-new-dev.allizom.org/report/list?range_unit=days&version=Firefox%3A16.0.2&range_value=7&signature=http%3a%2f%2fnetsparker.com%2fn%3f%00.php&page=11 from commment 1 is still throwing a 500; reopening.

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

Dev wasn't updating, and rhelmer brought it back up to the latest/master, now this WFM:

http://crash-stats-new-dev.allizom.org/report/list?range_unit=days&version=Firefox%3A16.0.2&range_value=7&signature=http%3a%2f%2fnetsparker.com%2fn%3f%00.php&page=11

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED.

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot