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

V2 legacy route for manifests is still using ArchiveLab infrastructure? #26

Closed
hadro opened this issue Sep 28, 2023 · 4 comments · Fixed by #74
Closed

V2 legacy route for manifests is still using ArchiveLab infrastructure? #26

hadro opened this issue Sep 28, 2023 · 4 comments · Fixed by #74
Assignees
Labels
bug Something isn't working High Priority

Comments

@hadro
Copy link
Collaborator

hadro commented Sep 28, 2023

See in this manifest, for example:
https://iiif.archive.org/iiif/2/1991-12-compute-magazine/manifest.json

All the images are being requested via the code on archivelabs, e.g.:
https://iiif.archivelab.org/iiif/1991-12-compute-magazine$44/full/full/0/default.jpg

And all the @ids are referencing Archivelab as well.

Is that as intended?

@glenrobson
Copy link
Collaborator

Two options:

  1. alias iiif.archivelab.org to iiif.archive.org and shutoff the labs service
  2. Change the manfiest 2 generation so that it creates iiif.archive.org IDs.

Currently the issue is that v2 is using the old image server not Cantaloupe.

@digitaldogsbody
Copy link
Collaborator

I think at the least we should change the v2 generation to use Cantaloupe for serving the images, even if we keep the archivelabs ids for everything else. This could also help with the second issue in #16

@glenrobson glenrobson added the bug Something isn't working label Jan 18, 2024
@benwbrum benwbrum added this to the Spring 2024 sprint milestone Jan 18, 2024
@digitaldogsbody digitaldogsbody self-assigned this Mar 7, 2024
@glenrobson
Copy link
Collaborator

Agreed to go with option 2.

@glenrobson
Copy link
Collaborator

Complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working High Priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants