-
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
Xhamster photo galleries stopping download at 24 images. #1777
Comments
I started to have the same problem about 2 days ago, tried several galleries, deleting cached files with same result, home it gets fixed on next release. |
@Je-Te-Ma I think I've fixed the issue. It's not official though. In the meantime, if you want, you can use this unofficial jar if you like. |
@mukulj77 patch works fine for me. I no longer face the 24 image limit using his fork. He seems to have forgotton to release his version? Not sure how the release works but I checked out his repo and compiled. The jar file I got was 27 MB, a bit larger then the expected 4 MB, but is what I got with @Je-Te-Ma The patched file can be download here: https://github.com/ripdave/ripme/blob/master/ripme-1.7.93-jar_patch_mukulj77.zip To satisfy Github.com, I had to zip the file to keep it under 25 MB. Thanks a lot to @mukulj77 for the fix and to the maintainers of this project |
First off, I am a COMPLETE novice here. I too have been getting limited at 24 pictures. So I uninstalled RIPME from my comp, restarted, downloaded the patch via the link in @ripdave 's post and then tried to rip another large gallery... Still getting stuck at the 24 pic limit. I'm new to ripme, and github, Im certainly not versed in programming etc. So is this a issue that is being worked on? Has it been fixed? Or is it still messing up for some folks out there? Is there any OTHER way to go about correcting this issue? |
yeah, it was working fine yesterday but went back to dying at 24 images this morning. |
@vindicator1 , you are right. It's happening again. I'll look into it next week. Bit swampped with things. |
@mukulj77 It seems that the pagination part that the ripper is searching for is now somehow obfuscated by the website. When opening the page source in Firefox, you get a There might be workarounds to get all pages: one could be to take the total number of images of the gallery, divide by 24 and round up, then loop through the pages (a bit messy, though). Another could be to make use of the |
That makes lot of sense. I'll definitely look into those ideas. |
@fuzzycheerio No, that is expected. The ripper looks at the mobile website, which does display 24 images per page. |
@naamval , ah gotcha. Thanks so much for taking a deeper dive and explaining these things to me, a admitted noob. =) |
This is an expansive change. Added the fasterxml.jackson package to POM for parsing the JSON object. TO-DO: Need to check the test cases.
@naamval The problem lied in quite a substantial revamp by the website. I think they are now dynamically generating the next pagination markup. tl;dr folks, please check out the temporary jar I've created for now. Would liase with the folks maintianing the project to see if they accept my solution. The jar is with all the dependencies, and hence large. |
@mukulj77 thank you for fixing this! I just tested 1.7.93 on an xhamster gallery with 149 images. It worked perfectly. This 24 image limit has been bugging me for a long time. Thank you! |
Just improving the bug-fix. TO-DO: Need to check the test cases.
@mukulj77 Thanks for your efforts to fix this issue! I ran a bunch of tests and it's working flawlessly. |
Expected Behavior
Expecting all 52 images of the gallery to be ripped.
Actual Behavior
Only 24 images are downloaded before it reports it as complete despite the gallery containing more images. Many galleries I have tried from Xhamster photo have had the same problem.
The text was updated successfully, but these errors were encountered: