Support JPEG Seadragon tile URLs - #980
Conversation
Deploying dezoomify with
|
| Latest commit: |
03989fd
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d649b28a.dezoomify.pages.dev |
| Branch Preview URL: | https://ophir-lojkine-modern-proxy-u.dezoomify.pages.dev |
lovasoa
changed the title
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 03989fd883
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| /bibliotheques-specialisees\.paris\.fr\/ark/, | ||
| /nla\.gov\.au\/nla\.obj.*\/view$/, | ||
| /_files\/\d+\/\d+_\d+\.(?:jpg|png)$/, | ||
| /_files\/\d+\/\d+_\d+\.(?:jpe?g|png)$/, |
There was a problem hiding this comment.
When the caller supplies the old Chrome identity from the reported ADIF failure, functions/proxy.js:23 still forwards that identity upstream, so /proxy continues receiving the same 403 response. This change only adds .jpeg Seadragon discovery; update the shared proxy handler to send the modern user agent and cover that behavior with a proxy test.
AGENTS.md reference: AGENTS.md:L12-L12
Useful? React with 👍 / 👎.
1 participant
Seadragon single-tile discovery recognized
.jpgand.png, but not the equivalent.jpegsuffix.Accept
.jpegin both automatic URL detection and metadata URL recovery. The proxy continues to forward the caller user agent unchanged.Regression coverage automatically opens a
.jpegtile URL and resolves its DZI metadata.cd tests && npm testpasses all 18 tests.