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

Adds functionality for a use of IIIF Auth access token, while also re… #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mejackreed
Copy link
Contributor

…questing a resources info.json within manifestor.

Just a WIP using IIIF Auth

…questing a resources info.json within manifestor.
@nein09
Copy link
Collaborator

nein09 commented Nov 25, 2015

This is great! When #94 is in, we may want to move this functionality into the CanvasObject.

@iangilman
Copy link
Collaborator

Now that #94 is merged, do we want to work on integrating this change?

/cc @aeschylus

@nein09
Copy link
Collaborator

nein09 commented Dec 16, 2015

And where would we want to put it? To me, it looks like it should go in the CanvasObject, but I may be misunderstanding it.

@aeschylus
Copy link
Contributor

It should be build in as a utility function or some type of special request proxy. It's purpose is to notify the client (in this case iiifmanifestlayouts, but potentially host software, like sul-embed or Mirador) that the image resource is protected or has degraded access.

Here is the IIIF Auth spec. No need to read it yet at this point, but the motivation section will set some context. There is a set of steps that the user may want to go through to authenticate themselves or gain access to the best version of an image available to them. We do want to make this data/process accessible through the API, but this is a low priority at the moment, as currently SUL-Embed is doing this itself (although in a less granular way.

The highest priority now is to get the example page with the clipping/fading/positioning/etc. of multiple tilesources working, then to make sure we're correctly delaying info.json requests and rendering that correctly. The last step should be to separate out the renderer/animation more so it is more intelligible, and so the core can communicate state to containing components more precisely through events.

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.

4 participants