Zachary, do you mind attaching the add-on which generated this error? Thanks!
Created attachment 540077 [details] Full source of my add-on In this version of the source I've applied my fix to get past the validation failure.
Created attachment 540080 [details] XPI WITH Problem This XPI will create unexpected validation failure.
Worked fine for me on the command line: $ ./addon-validator firebomb-plugin.xpi Summary: ------------------------------ Detected type: Extension/Multi-Extension ------------------------------ Test failed! Errors: Warning: Jetpack module hash unknown Warning: Jetpack module hash unknown Warning: Hidden files and folders flagged
This may be an issue with the prod python setup. The validator JSON output from the bad xpi has the following error File \"/usr/lib/python2.6/json/scanner.py\", line 55, in iterscan rval, next_pos = action(m, context) File \"/usr/lib/python2.6/json/decoder.py\", line 68, in JSONNumber res = fn(integer) RuntimeError: maximum recursion depth exceeded while calling a Python object Output cleaned up a little to make readable
Status: UNCONFIRMED → NEW
Ever confirmed: true
My output was from khan which is supposed to mirror production
Component: Developer Pages → Add-on Validation
QA Contact: developers → add-on-validation
This problem should have been fixed when we switched from `json` to `simplejson`. If it hasn't been, then this is another reason why we need a streaming JSON parser.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Don't json and simplejson trivially parse JSON from streams? Or do you mean something SAX-like?
SAX-like. We need something that's not based on recursion.
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.