Ally Invest returning HTML with response.ok about allyinvest.py

[anthonymorast]

Hi, thanks for submitting this issue. First and foremost, I'm 100% OK with Pull Requests (apologies in advance if I don't get to them right away) I would really like to see this project grow and develop into something highly applicable.

As for the error, I believe I have seen this before. If I recall correctly, in my case Ally was throttling my API calls since I was submitting too many requests or something along those lines (I could be wrong as it's been a while).

Do you know what errors you're getting in the HTML are? From this project's perspective, I don't think there's much we can do besides check if we're getting JSON back and return an error (and the HTML) if not since it's unpredictable what Ally might send back from a request.

[wmorgansoftware]

So with the HTML, the contents are the "Site is experiencing issues" page that shows up if you go to Ally Invest with an internet browser and things are down. I reported it to them a while back because it wasn't JSON, and got the typical "We'll look into it" without follow-up. I agree the only thing we can do is a check to make sure that it's in JSON format, even if it's just a try/except on JSONDecodeError. They did do some adjustments over the weekend (probably from all the API call rejections happening last week, even the first request), so I want to see if the issue is OBE before making any changes.

I know there's a 60 call per minute limit. Also, on my interpretation, I'm using another source for prices because it's real-time, whereas Ally is on the delay. It's Yahoo so who knows what will happen; I still remember the whole debacle about people figuring out what the letter codes meant for the quote URL calls, and they went crazy and destroyed everything.