Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Market graph on wsj.com #2102

Closed
luixxiul opened this issue Jun 6, 2016 · 3 comments
Closed

Market graph on wsj.com #2102

luixxiul opened this issue Jun 6, 2016 · 3 comments

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Jun 6, 2016

From support: Market graph on wsj.com does not work,

Confirmed on http://www.wsj.com/asia. Disabling Bravery shields does not help. On console:

POST http://api.wsj.net/api/michelangelo/timeseries/history?hash=452753711&ckey=57494d5ed7 400 (Bad Request)
asia:1 POST http://api.wsj.net/api/michelangelo/timeseries/history?hash=452753711&ckey=57494d5ed7 400 (Bad Request)
nr-943.min.js:1 Uncaught TypeError: Cannot read property 'api' of undefined
@bbondy bbondy added this to the 0.10.4dev milestone Jun 9, 2016
@bbondy
Copy link
Member

bbondy commented Jun 13, 2016

This is fixable by calling cb({}) inside session.webRequest.onBeforeSendHeaders(function (details, cb).
For some reason even if you call cb({ requestHeaders: details.requestHeaders }) it causes the graph to not show up. I'm not sure what would be different because of this though. I guess the callback is causing something else different, maybe like the HTTP Method or something else to also change. cc @diracdeltas @bridiver

@bbondy bbondy modified the milestones: 0.10.5dev, 0.10.4dev, 0.11.0dev Jun 13, 2016
@bbondy bbondy removed this from the 0.11.1dev milestone Jul 13, 2016
@bridiver
Copy link
Collaborator

fixed by brave/muon@5f11d53

@luixxiul luixxiul added this to the 0.11.6dev milestone Aug 16, 2016
@bbondy
Copy link
Member

bbondy commented Aug 16, 2016

mystery solved 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants