The initial toolkit commit throws away the API response if validation fails, but this will probably happen pretty often in the MVP development period. Let's instead be permissive, logging an error but otherwise attempting to return whatever we got back from the API fetch.

Backed out for xpcshell failures on test_product.js

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=3f589713e266401f6f89f8547d79109824e55ec7&selectedTaskRun=Hwo3HglYRW2KOCyP3iMzgg.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=421100887&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/c0d3d8606d296a2f62b90cc11f8fb506f12f2b3d

sorry for breaking stuff during code freeze! I thought I canceled this on Lando before it made its way to autoland :-\

No worries Jared, if you fix it and still want it in 116 you can give it another go.

unassigning myself, as :fchasen is going to fix this in another patch he's working on.

This was closed by the patch for bug 1840678