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

wallabag can't retrieve contents for this article: theatlantic.com #2075

Closed
Strubbl opened this issue May 13, 2016 · 6 comments
Closed

wallabag can't retrieve contents for this article: theatlantic.com #2075

Strubbl opened this issue May 13, 2016 · 6 comments

Comments

@Strubbl
Copy link
Contributor

Strubbl commented May 13, 2016

Issue details

http://www.theatlantic.com/international/archive/2013/10/behind-the-bad-indian-coder/280636/
http://www.theatlantic.com/technology/archive/2013/06/fix-things-never-force-it-lessons-from-grandpa/276873/

Content is not loaded.

Environment

  • wallabag version (or git revision) that exhibits the issue: 2.0.4
  • How did you install wallabag? Via git clone or by downloading the package? git clone
  • Last wallabag version that did not exhibit the issue (if applicable):
  • php version: 5.6.20
  • OS: Debian Jessie
  • type of hosting (shared or dedicated): dedicated
  • which storage system you choose at install (SQLite, MySQL/MariaDB or PostgreSQL): MySQL
@Strubbl
Copy link
Contributor Author

Strubbl commented Jun 15, 2016

a workaround is to add the page www.theatlantic.com/amp/article/276873/

@j0k3r can this be automated since the url is always the same except the article ID in the end? Maybe with the single page link stuff? I will try

@Strubbl
Copy link
Contributor Author

Strubbl commented Jun 15, 2016

Yes!!! single_page_link is the solution.

@Strubbl
Copy link
Contributor Author

Strubbl commented Jun 15, 2016

should be fixed with j0k3r/graby-site-config#11

@j0k3r
Copy link
Member

j0k3r commented Jun 16, 2016

I closed your PR on my fork because it's better to submit PR about siteconfig to the original repo (https://github.com/fivefilters/ftr-site-config) :)

@solarchemist
Copy link

solarchemist commented Jun 17, 2016

This is still an issue for all articles from theatlantic.com (though not all, not sure what differs).

This fails:
http://www.theatlantic.com/entertainment/archive/2015/06/clear-the-kitchen-table-how-apple-and-ibm-marketed-the-first-personal-computers/396047/

Whereas this manual fix (as per suggestion above) works:
http://www.theatlantic.com/entertainment/amp/article/396047/

Minor side-effect of the work-around: no more title graphics. Whereas using the "original" long URL always imports nice title graphics, the shorter URL consistently does not.

Installed via git clone, 2.0.5, PHP 5.6.22, Debian Jessie 8.5, dedicated VPS, MySQL.

@j0k3r
Copy link
Member

j0k3r commented Jun 17, 2016

@chepec this will be fixed in the next wallabag release (see fivefilters/ftr-site-config#171).
Feel free to put a comment here if it's not the case.

@j0k3r j0k3r closed this as completed Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants