-
Notifications
You must be signed in to change notification settings - Fork 6
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
The Processor now requires URLs in a specific format #30
Comments
Thanks for bringing this issue to my attention. I'll take a look at it and see if I can come up with a solution. It might be as easy as removing the beginning-of-string anchor ( |
I encountered this as well now. It would be great if this was more flexible. We plan to use |
Thanks; I'll try to get to this soon. |
@tarjelavik The pull request referenced above should help once it's merged – you would pass Are you using the package directly, or through serverless-iiif? If it's the latter, it'll take a little bit for the feature to work its way up through the stack. |
@mbklein Great! I tested the commit in my repo and it works :-). If i used No, i can't use the |
That is fantastic! Is your Azure version open source/hosted anywhere? I'd love to combine our efforts and maybe create a deployable Azure version that's linked from the main documentation site. |
Great :-). It is hosted here: https://git.app.uib.no/uib-ub/cultural-heritage-collections/serverless-iiif-azure. It is my first attempt at writing for Azure, and a lot was done with AI help. I don't think it is implemented very well, but it work well locally. It has not been tested on Azure, so the bicep IaC files are more a placeholder. I hope we can get help from our IT-department with the next steps. A link from It would be great is your repo could support Azure as well, and given your experience with running it on in the cloud it would be a big help! |
Hi all, I found an interesting issue while using the module.
Configuration
I'm using the latest version of the iiif-processor module:
4.0.4
.Problem
I got an error when creating a new
Processor
object passing a URL with this pattern:https://<domain>/.*/iiif/2/.*
.It seems that it is no longer possible to use this kind of pattern.
Going deeper, I see that a regex requires that between the domain and the
/iiif/2
part there are no extra strings here https://github.com/samvera/node-iiif/blob/v4.0.4/src/processor.js#L19.The text was updated successfully, but these errors were encountered: