Fixed Motherless ripper

[brantspar]

The Motherless ripper was broken as the rip operation was not waiting on all the threads created to download the files from motherless.

This repo reminded me as to why OO inheritance is just plain bad... too hard to follow what's happening!

Also

• increased default http retries to 3 (why only 1?)
• fixed deprecation message for Motherless ripper

Category

This change is exactly one of the following (please change [ ] to [x]) to indicate which:

• a bug fix (Fix #...)
• a new Ripper
• a refactoring
• a style change/fix
• a new feature

Testing

Required verification:

• I've verified that there are no regressions in mvn test (there are no new failures or errors).
• I've verified that this change works as intended.
  • Downloads all relevant content.
  • Downloads content from multiple pages (as necessary or appropriate).
  • Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
• I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

• I've added a unit test to cover my change.

[brantspar]

@soloturn for visibility

[brantspar]

I have fixed the tests however the build ends with this error

Could not create new tag "refs/tags/latest-194-merge" (Resource not accessible by integration) therefore updating existing tag "tags/latest-194-merge"
  Error: Resource not accessible by integration

It looks like it's related to this -> marvinpinto/actions#61 -> The pull requests run with read-only permissions because they are made by other users