-
Notifications
You must be signed in to change notification settings - Fork 626
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
Code Health: Audit Functionality of Existing Rippers (Help Wanted) #2058
Comments
Note that at a point in the past I opened issues for missing test files and created stub test files, which maybe just hid the problem. Here are the issues I referenced at that time (created to track the work of needing to add confirmation). #915, #916, #917, #918, #919, #920, #921, #922, #923, #924, #925 |
For Yuvutu, it seems the site has changed enough that ripping videos is broken (see VideoRippersTest) but ripping galleries is fine.
|
FivehundredpxRipper functionality inconclusive. The test URL doesn't seem to exist. There are many possible locations but we need a new test case basically to check it out. |
FlickrRipper functionality inconclusive. Navigating to the website gives 403. Maybe needs to be logged in, maybe ripper needs cookies.
|
FurAffinity functionality inconclusive. I waited around a minute and it kept doing 1000 ms sleeps but didn't log anything else to give me an idea of what it might be doing that's taking so long. Here's the output I got in the window though:
|
Many of the rippers in the app are really out of date. Either sites have changed so that the rippers no longer work, or sites have even gone offline entirely.
Check the boxes as rippers are verified working. Check the "not working" box and comment the error if they are not.
The text was updated successfully, but these errors were encountered: