-
Notifications
You must be signed in to change notification settings - Fork 24
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
6 MB fallback fails for IIIF IDs that have a % in them (encoded /) #52
Comments
@tpendragon Can you provide steps to reproduce this issue? Even though you have a workaround, I'd like to fix it in core, but I've forgotten exactly what the issue is. |
I think you just have to put a file in an s3 bucket with a folder and then try to render it |
A large file that has to fall back to cache |
I believe this was fixed by samvera/node-iiif#23 (via 84da4d6). I'm closing this issue but @tpendragon if you deploy the latest version and find that this is still an issue, feel free to reopen. |
We had to fix this by installing a viewer-request function (https://github.com/pulibrary/serverless-iiif/blob/main/src_edge/index.js), the comment in there describes the issue best I think:
The text was updated successfully, but these errors were encountered: