Cache does not take provider config into account

[florian-lefebvre]

🐛 The bug

Currently, resolveFont for each provider is cached using a key that is partially made of the family options hash. That's great but if you use some providers options then change them, cache is not invalidated.

🛠️ To reproduce

https://stackblitz.com/edit/github-pkgkk2lt?file=cache.js

🌈 Expected behaviour

Take the provider options into account, when applicable (ie. only for google providers atm).

ℹ️ Additional context

I'd be happy to submit a PR. Note there's overlap with #183 since the proposed design would solve this issue entirely

[sushichan044]

Hello, I’m running into the same issue.
Ref: #240

Instead of changing the structure of resolveFont,
I’m considering adding an internal API that generates a safe cache key by default, including the provider name and provider options. ( draft impl: #239 )

What do you think?

[sushichan044]

@florian-lefebvre
When you have time, could you please review the following issue and let me know your thoughts?