Page MenuHomePhabricator

Add caching of citoid results in the extension
Open, Needs TriagePublic

Description

This doesn't help with third party usage of the API, but in conversation with @Krinkle today he suggested it might be a good idea to cache the citoid results to prevent users from making too many requests in rapidity.

We could add caching in the back end too but that is blocked by T361576

Event Timeline

Is this for purely client-side cache (in-memory, just affecting people who repeatedly click "generate" in a single session)? If so it seems harmless, but I can't imagine it'd have much impact.

Is this for purely client-side cache (in-memory, just affecting people who repeatedly click "generate" in a single session)? If so it seems harmless, but I can't imagine it'd have much impact.

Yup. I agree it probably won't have major impact, but for sites that do 429s for repeated requests in quick succession, it might help?