-
Notifications
You must be signed in to change notification settings - Fork 10
feat: Add Makefile target to generate reader.archive.org URLs with OP… #140
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
feat: Add Makefile target to generate reader.archive.org URLs with OP… #140
Conversation
…DS feeds and enable manifest route forcing for the reader service.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR enables the manifest route in Thorium Web reader (which is disabled by default in production) and adds a convenient Makefile target for generating reader.archive.org URLs with OPDS feeds.
- Set
MANIFEST_ROUTE_FORCE_ENABLE=trueto enable the/read/manifest/route in production - Add
make urlcommand to generate reader.archive.org URLs from cloudflared tunnel URLs - Configure the environment variable in both Dockerfile and compose.yaml
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docker/reader/Dockerfile | Adds ENV variable to force-enable manifest route at build time |
| compose.yaml | Adds runtime environment variable for manifest route enablement |
| Makefile | Implements new url target that extracts tunnel URL, constructs OPDS feed URL, and generates reader.archive.org link |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…arguments in Dockerfile and compose.yaml.
|
ronibhakta1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks for contributing to this issue.
Problem
Latest Thorium Web (main branch) disables the /read/manifest/route in production by default for security.
Solution
Set MANIFEST_ROUTE_FORCE_ENABLE=true in the Dockerfile to enable it.
Screen.Recording.2025-12-16.at.02.18.37.mov
make url
Add a make url command to the Makefile.
Stakeholders
@mekarpeles @ronibhakta1