-
Notifications
You must be signed in to change notification settings - Fork 630
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
Reddit: (Link domain support) Some links in comments are not handled correctly causing them to not be downloaded #90
Comments
As far as getting 1 MP4 and 1 JPG went it worked fine:
Window$ 7 Ultimate SP1 x64 |
@rautamiekka, the intended downloads are You've downloaded |
^ In which case it's a bug, possibly a 'website changed' one, not fully sure. |
Does this happen with links to other sites, or just erome? |
@cyian-1756, I reviewed my logs and there's lots of other sites, yeah. Two days ago there were 71 instances where an Then for .gifv links, there's 211 of them following For the close paren, I think ripme gets json formatted by reddit, which would include the markdown formatting, which for links is For imgur's gifv, I'd imagine it's because it doesn't match the "Direct link to image" pattern in RipUtils.java. |
Expected Behavior
The link within the comment should be parsed normally and download.
Actual Behavior
Examples, including full command output:
and
This is on a completely new test setup with no previous downloads or configuration, everything is default settings.
In both of these examples the parent post is downloaded (that's fine) but the content linked in the comment is not. I figured when I saw the trailing paren in the first one it wasn't being escaped or parsed properly, which is why I looked for a second example that also used markdown formatting, but in the second example the imgur .gifv looks like it's formatted just fine in the error output but it also doesn't work so I'm not sure and that's why I included both examples. The more I review this the more I think it's possible that I just stumbled on two separate issues at the same time, but I'm not entirely sure at this point.
Also I probably should mention that normally I'm using this as part of a entire user download run with a cronjob and not a single comment, however I noticed these in my logs and tailored my examples so they were just related to the issue and hopefully easier to reproduce.
The text was updated successfully, but these errors were encountered: