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

Serve Gottschalk images from prod server #826

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

dchiller
Copy link
Collaborator

This PR updates the CU-hosted manifest for the Gottschalk reconstruction to serve images from production rather than staging. With Release 3.2-0.12.0 (https://github.com/DDMAL/cantus/releases/tag/v.3.2-0.12.0) these images are on prod, so we can serve them from prod for both staging and production purposes.

Copy link

@jacobdgm jacobdgm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not at all an expert IIIF manifests, but if the change is to replace staging.simssa.ca with simssa.ca in this entire file, I trust your text editor to have not missed anything. So I'll give my approval for now, but I do think @ahankinson's suggestion makes sense, so probably update that before merging.

@ahankinson
Copy link
Member

Also, visiting https://cantus.simssa.ca/iiif/lacuna.jpg redirects you to https://cantaloupe/iiif/2/lacuna.jpg/info.json so there's something up with the server configuration that needs to be fixed.

@dchiller
Copy link
Collaborator Author

Also, visiting https://cantus.simssa.ca/iiif/lacuna.jpg redirects you to https://cantaloupe/iiif/2/lacuna.jpg/info.json so there's something up with the server configuration that needs to be fixed.

There is nothing on the other end of this (ie. there is no info.json resource for the lacuna image -- or for any image we are serving). I'm hesitant to put in the effort for this given that this is an interim solution and #822 is for putting in place something long-term (probably with greater collaboration with the fragmentarium folks). Does it seem important that this happen now?

@dchiller
Copy link
Collaborator Author

The latest commit addresses Andrew's comment. I'm going to merge to make sure at least this change works, but let's continue the discussion.

@dchiller dchiller merged commit efd9adc into DDMAL:main Feb 14, 2024
2 checks passed
@dchiller dchiller deleted the update-gottschalk-manifest-for-prod branch February 14, 2024 13:06
@ahankinson
Copy link
Member

There is nothing on the other end of this (ie. there is no info.json resource for the lacuna image -- or for any image we are serving)

Then it's not IIIF. The base of the URI must resolve to an info.json response -- that's how image viewers find out how many zoom levels there are, tile sizes, etc.

Section 5: https://iiif.io/api/image/3.0/#5-image-information

Servers MUST support requests for image information. The response is a JSON document that includes technical properties about the full image. It may also contain rights information, and services related to the image.

Canteloupe should generate the info.json automatically, so you don't need to write it by hand.

You have a canteloupe response that is looking for the info.json, but it's not configured correctly. Changing it to https://cantus.simssa.ca/iiif/2/lacuna.jpg/info.json almost works too, but something in the URL rewriting is adding an extra /2/ in there.

@dchiller
Copy link
Collaborator Author

Canteloupe should generate the info.json automatically, so you don't need to write it by hand.

Ah, ok. Then yes, just a configuration issue. The extra 2 is nginx. I'm sure the redirect is a default configuration in cantaloupe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants