Broken API
shweshi opened this issue · comments
curl https://terminal-stocks.dev/market-summary is returning.
Sorry, we couldn't find. Please check the stock ticker and provide correct one. Invalid Crumb. For Developers - https://bit.ly/yahoo-finance-api-feedback
Same for other APIs with stock tickers.
@chris-hmmr looks like something has changed from Yahoo API.
Hi @shweshi, yes, it seems Yahoo is curtailing API access. I shall investigate if there is a way to circumvent or fix this.
Looks like the issue on the dependent library has been resolved, see here. I'll update the package and check locally if that fixes our issue as well.
Hi @shweshi, I opened a PR with a fix https://github.com/shweshi/terminal-stocks/pull/20/files. Please review.
@chris-hmmr merged thanks.
@shweshi you're welcome. Bugfix is now live terminal-stocks.dev/AAPL
![image](https://private-user-images.githubusercontent.com/1536058/323608020-bcd5851e-d6ea-49f7-81c4-5385a818a2d1.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQxMzY1NDMsIm5iZiI6MTcyNDEzNjI0MywicGF0aCI6Ii8xNTM2MDU4LzMyMzYwODAyMC1iY2Q1ODUxZS1kNmVhLTQ5ZjctODFjNC01Mzg1YTgxOGEyZDEucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgyMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MjBUMDY0NDAzWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MjE5YmVmN2ExMzdmOTI3MTc0ZWMwNDEzZTQ5OWE4NGE1MzJmNjEyNDhlNmJmNDUyYWVjYmUyY2JkYzhiY2NjOSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.3P2fIj4OoJnhZ-SfRRuzeMCT6Ewurr3JOnzXQsZDQjw)