Skip to content
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

Rebuild of Reality Lovers Fixes #1851 #1864

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pops64
Copy link
Contributor

@pops64 pops64 commented Oct 15, 2024

API appears to have been abandoned and now returns 403 errors. This new scraper now relies on HTML for its data. All works. Title and Cover are both grabbed from index page. These seem to be the most accurate and useful data source. On single scene scrape we fall back to using the gallery and the URL for these sources

API appears to have been abandoned and now returns 403 errors. This new scraper now relies on HTML for its data. All works. Title and Cover are both grabbed from index page. These seem to be the most accurate and useful data source. On single scene scrape we fall back to using the gallery and the URL for these sources
@pops64 pops64 closed this Oct 16, 2024
@pops64 pops64 reopened this Oct 16, 2024
@pops64
Copy link
Contributor Author

pops64 commented Oct 16, 2024

I am going to leave this open. But the API still works(as of 10/14/2024 mabye, 10/16/2024 was getting nothing but 404 codes) but the scraper code is funky and has logic flaws in it. If wanted I could fix the logic flaws in the OG reality lovers scraper but not really a fan of the 404 status codes and when using curl or a browser to the same address as the scraper you get a access denied message. And I don't see any calls to this API on there front end

@pops64 pops64 changed the title Rebuild of Reality Lovers Rebuild of Reality Lovers Fixes #1851 Oct 16, 2024
@pops64
Copy link
Contributor Author

pops64 commented Oct 30, 2024

Per #1851 the current implementation of this is broken for some users. This should be merged in with the next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant