karlwancl / YahooFinanceApi

A handy Yahoo! Finance api wrapper, based on .NET Standard 2.0
MIT License
158 stars 58 forks source link

Fixed too many requests error caused by missing user agent header #60

Closed tranb3r closed 1 year ago

tranb3r commented 1 year ago

Fixes #59

sergemat commented 1 year ago

@karlwancl please increment version number and push new nuget package for this PR.

karlwancl commented 1 year ago

@sergemat @tranb3r Thanks for the fix. Version bumped to 2.3.0 and deployed to nuget.

In the 1st run, the pipeline has failed. I re-run it and it succeed. I have rerun the tests on my local a few times and find it happens occasionally, not sure if it is another bug when communicating with Yahoo! Finance https://github.com/karlwancl/YahooFinanceApi/actions/runs/5774266592/job/15651047151