ripme icon indicating copy to clipboard operation
ripme copied to clipboard

Fix XhamsterRipper by getting cookies first

Open ehump12000 opened this issue 5 years ago • 1 comments

Category

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

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

Description

The Xhamster ripper will now set its cookies from the first page similar to EightmusesRipper.

Testing

Required verification:

  • [x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
  • [x] I've verified that this change works as intended.
    • [x] Downloads all relevant content.
    • [x] Downloads content from multiple pages (as necessary or appropriate).
    • [x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
  • [x] 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.

ehump12000 avatar Mar 13 '20 04:03 ehump12000

Resolved conflicts. Will test and merge later.

metaprime avatar Jan 07 '25 21:01 metaprime