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

Fix #1118 - Update FanFictionParser.js #1119

Merged
merged 1 commit into from
Nov 26, 2023

Conversation

Kiradien
Copy link
Collaborator

Fix for loading inconsistent caching issue on ff.net; automatically checks secondary link upon any failure to retrieve chapter or read chapter text. Separate link calls info directly without interacting with broken caching. Uses existing throttle logic for second call to avoid CloudFlare detection.

Note: The detection could be narrowed down more to detect page layout and specific error message; This is not done because this method does actually resolve several other rare chapter load issues that have occurred in the last year.

Fix for loading inconsistent caching issue on ff.net

Update FanFictionParser.js

Removed shortUri double-negate; code will run functionally equivalent.
@Kiradien
Copy link
Collaborator Author

This is a fix for bug #1118 - not sure why it didn't link correctly; must have been a check-in issue.

@Kiradien
Copy link
Collaborator Author

Note regarding this fix, if too many chapters have the caching issue in a short timeframe it can still result in temporary failures. It's not a perfect fix, but there are no perfect fixes when the issue is serverside. The above issue can be fairly effectively resolved by setting 30 seconds per chapter if it crops up.

@dteviot dteviot merged commit 3b67a9c into dteviot:ExperimentalTabMode Nov 26, 2023
1 check passed
@Kiradien Kiradien deleted the patch-2 branch November 26, 2023 20:38
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.

2 participants