Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'cacheBust' query param when fetching NQ #241

Closed
donaldgray opened this issue Jan 23, 2025 · 0 comments · Fixed by #274
Closed

Add 'cacheBust' query param when fetching NQ #241

donaldgray opened this issue Jan 23, 2025 · 0 comments · Fixed by #274
Assignees
Labels
refined Identify refined tickets in backlog SoW5 SoW5

Comments

@donaldgray
Copy link
Member

donaldgray commented Jan 23, 2025

IIIF Presentation uses Protagonist named-queries to generate content-resources for assets on a manifest. If there are frequent changes to a manifest we could be making a number of calls for named-queries in a short space of time. We could run into issues if we are served a cached/stale version of a manifest as it could be rendering out of date assets.

We are filtering by batch-id to images can't be added but they could change if:

  • A failing image is successfully reingested
  • A successful image has been reingested and failed
  • An origin changes and the source image is a different size
  • A policy, e.g. thumbnail policy, changes meaning different sizes are available.

Some of these are more likely than others but to avoid potential issues we should append some sort of cache-busting query parameter to the request as the Protagonist cache varies by query-param. This should be random, suggested ?cacheBust={DateTime.UtcNow.Ticks}.

@donaldgray donaldgray added SoW5 SoW5 refined Identify refined tickets in backlog labels Jan 23, 2025
@donaldgray donaldgray changed the title Add 'cacheBust' header when fetching NQ **Unrefined** Add 'cacheBust' header when fetching NQ Jan 24, 2025
@donaldgray donaldgray self-assigned this Feb 14, 2025
@donaldgray donaldgray changed the title Add 'cacheBust' header when fetching NQ Add 'cacheBust' query param when fetching NQ Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refined Identify refined tickets in backlog SoW5 SoW5
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant