Closed Bug 395988 Opened 17 years ago Closed 14 years ago

JSON error alert on loading page (caused by bad browser detection that only detects "Firefox")

Categories

(Tech Evangelism Graveyard :: English US, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: abillings, Unassigned)

References

()

Details

If I load http://my.break.com/media/view.aspx?ContentID=359313 in trunk, I receive an alert from the page with the message: "error parsing JSON . - SyntaxError: unterminated string literal" This does not happen in 2.0.0.6 and nothing shows up in the error console. It might be some custom script running on the page. This was found in Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a8pre) Gecko/2007091204 Minefield/3.0a8pre. Dveditz, please add who you wish to this. I don't have a list of everyone's e-mail addresses that you had mentioned. :-)
This is a problem with the code on the site relying on having "Firefox" in the user agent string. The postDynamicStatsCallback function at http://media1.break.com/static/app/v1/global/js/dynamicstatscontent.js tries to parse a large amount of JSON from an XML node's value, but only calls "normalize()" on the XML doc to collapse adjacent text nodes (bug 194231) if the UA contains "Firefox". This isn't true in Minefield, so it tries parsing only the first 4k of JSON and fails. Spoofing my Minefield UA to include "Firefox" fixes the problem.
Assignee: nobody → english-us
Component: General → English US
Product: Firefox → Tech Evangelism
QA Contact: general → english-us
Version: Trunk → unspecified
Summary: JSON error alert on loading page → JSON error alert on loading page (caused by bad browser detection that only detects "Firefox")
Blocks: geckoisgecko
INCOMPLETE due to lack of activity since the end of 2009. If someone is willing to investigate the issues raised in this bug to determine whether they still exist, *and* work with the site in question to fix any existing issues, please feel free to re-open and assign to yourself. Sorry for the bugspam; filter on "NO MORE PRE-2010 TE BUGS" to remove.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → INCOMPLETE
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.