Added referer to avoid 405 error in downloads

[89am89]

Category

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

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

Description

Erome ripper returned error 405 trying to download images and videos due to the absence of its referer property. Adding the website referer string to the ripper fixed the issue.

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.

[soloturn]

hey thank you! does this: https://github.com/ripmeapp2/ripme/issues/47 fixed here: https://github.com/ripmeapp2/ripme/commit/d8cd6291a5c32f997e05717fa396d8164b291def and released here: https://github.com/ripmeapp2/ripme/releases work for you as well?

[89am89]

Hi, yes it seems to be the same fix. Sorry, I didn't know there was a 2.0 repository. Can I ask you why you created it instead of continuing with the old version? I did the fix for the old version but I think many users like me didn't know about the existence of the new repo.

Thanks

Il giorno lun 27 dic 2021 alle ore 11:36 soloturn @.***> ha scritto:

hey thank you! does this: ripmeapp2#47 https://github.com/ripmeapp2/ripme/issues/47 fixed here: @.*** https://github.com/ripmeapp2/ripme/commit/d8cd6291a5c32f997e05717fa396d8164b291def and released here: https://github.com/ripmeapp2/ripme/releases work for you as well?

— Reply to this email directly, view it on GitHub https://github.com/RipMeApp/ripme/pull/1938#issuecomment-1001499552, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXBUDN5FAWKOJM6I3EOI42LUTA6TVANCNFSM5KYY2EVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

[soloturn]

people with merge rights abandoned it a little. i took it as a learning how to do automated build and release in github and created pull requests here.

[cyian-1756]

@soloturn Do you want merge rights in the main repo? If so either I or @metaprime will add you