mirror of
https://github.com/go-shiori/shiori.git
synced 2026-04-25 06:25:54 +03:00
[GH-ISSUE #55] Cache not working? #34
Labels
No labels
component:backend
component:builds
component:builds
component:extension
component:frontend
component:readability
database
database:mysql
database:postgres
database:sqlite
feature:ebooks
github_actions
good first issue
hacktoberfest
note:duplicate?
note:fixed?
note:out-of-scope?
os:windows
priority:high
priority:low
pull-request
resolution:as-intended
resolution:cant-reproduce
resolution:duplicate
resolution:fixed
resolution:wontfix
tag:TBD
tag:big-task
tag:help-wanted
tag:huge-data
tag:meta
tag:more-info
tag:next
tag:no-stale
tag:requires-migrations
tag:research
tag:security 🛡️
tag:stale
tag:waiting-for-assignee
type:bug
type:documentation
type:enhancement
type:meta
type:ux
user:cli
user:web
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/shiori#34
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @tomas on GitHub (Mar 6, 2018).
Original GitHub issue: https://github.com/go-shiori/shiori/issues/55
Hi, just installed shiori on a server and trying it out. Looks great!
Unfortunately, I added a few bookmarks and none of them generated any cached versions. In what conditions does this happen? Is there some dependency that I have to install on my server?
Thanks
@RadhiFadlillah commented on GitHub (Mar 7, 2018):
Hi @tomas, thanks for using this app and reporting this issue.
No, there are no dependency needed.
Normally,
shiorishould fetch the content by default. May I know how you add the bookmark ? Using command line, web interface or usingshiori import? If possible, could you give me the URL whereshioridoesn't fetch any content ?Thank you.
@tomas commented on GitHub (Mar 7, 2018):
Via the web interface, although I did try updating the link afterwards from the command line.
One of the links I tried was this small HTML page:
https://tinyapps.org/system.html
@RadhiFadlillah commented on GitHub (Mar 8, 2018):
Hi @tomas, could you check it again ?
The caching process is done by go-readability, and I've push the fix at 9724ab.
@tomas commented on GitHub (Mar 8, 2018):
It works! However, links in the copied HTML are all wrong (they're somehow prepending the original link's hostname in them):
@RadhiFadlillah commented on GitHub (Mar 8, 2018):
Fixed in go-readability#558d9b4.
@tomas commented on GitHub (Mar 8, 2018):
Great, thanks.
Not related, but, would it be possible to have a bookmark added just by entering a URL and hitting the enter key? Having to use the mouse and click the "Done" button affects usability IMHO.
@tomas commented on GitHub (Mar 8, 2018):
Oh and it looks like go-readability's content lookup doesn't work always. Here's another URL where I get an empty result: https://www.plop.at/en/bootmanager/index.html
@RadhiFadlillah commented on GitHub (Mar 8, 2018):
Yeah, it's in the future plan.
Yeah, it's still in development and I haven't finished porting the entire methods from
mozilla/readability. There are some parts of code that I find quite hard to port, especially since Go and JS have opposite character (static vs dynamic language), so PR and contributions are always welcome.BTW, since
go-readabilityis port ofmozilla/readability(which powers reader mode in Firefox), that means if Firefox unable to render the readable format for a website, most likelygo-readabilitycan't do it as well.@tomas commented on GitHub (Mar 8, 2018):
I see. golang isn't my weapon of choice, so unfortunately I cannot help you much with PRs in
go-readability. Thanks again!