gadicc / node-yahoo-finance2

Unofficial API for Yahoo Finance
https://www.npmjs.com/package/yahoo-finance2
MIT License
353 stars 58 forks source link

It looks like quoteSummary now needs crumb #686

Closed Yhprum closed 9 months ago

Yhprum commented 9 months ago

Bug Report

Describe the bug

https://query2.finance.yahoo.com/v10/finance/quoteSummary/AAPL now requires a crumb (+cookie), and the module give the Quote not found for ticker symbol error.

Minimal Reproduction

await yahooFinance.quoteSummary("AAPL")

Environment

Browser or Node: Node Node version: v18.15.0 Npm version: 9.5.0 Library version: ^2.7.0

z3nful commented 9 months ago

Its working for me under the same conditions, are you using a proxy that could be blocking the cookie call?

const result = await yahooFinance.quoteSummary("AAPL")
console.log(JSON.stringify(result, null, 2));

Fetching crumb and cookies from https://finance.yahoo.com/quote/AAPL... Success. Cookie expires on Thu Sep 26 2024 00:48:51 GMT-0700 (Mountain Standard Time) New crumb: MMGROexZnaZ { "summaryDetail": { "maxAge": 1, "priceHint": 2, "previousClose": 176.08, "open": 174.82, ... }

Yhprum commented 9 months ago

False alarm, it looks like it was a different error - I was searching for https://query1.finance.yahoo.com/v10/finance/quoteSummary/AAPL231013C00100000?crumb=mycrumb&modules=price,summaryDetail

which has the put option at that strike/expiry but not the call, so the error was from it just not being there. Sorry bout that and thanks for the confirmation!

gadicc commented 9 months ago

Thanks, @z3nful, and @Yhprum for reporting back!